Skip to content

RFC: Schematics: Extend from @schematics/angular #882 #892

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 3 commits into from

Conversation

MichaelWarneke
Copy link
Contributor

Inherit the ngrx schematics from angular schematics.

The angular devkit has been updated to allow extends.
angular/devkit#34

By inheriting the hole package instead of some individual generators reduces boiler plate in the collection.json and makes sure that future angular generators are included automatically.

@coveralls
Copy link

coveralls commented Mar 6, 2018

Coverage Status

Coverage increased (+0.04%) to 93.575% when pulling 80b0e34 on MichaelWarneke:master into d1fbd35 on ngrx:master.

@brandonroberts
Copy link
Member

This would require a minimum version of the CLI with the devkit dependency correct?

@MichaelWarneke
Copy link
Contributor Author

Yes. The extends property got merged into devkit version 0.3.1 which got merged into angular cli 1.7.0-rc.0.
merged into cli

@brandonroberts
Copy link
Member

@MichaelWarneke can you add the following peer dependencies to the package.json for schematics to ensure there a minimum version that supports this for V6?

"@angular-devkit/core": "^0.4.0",
"@angular-devkit/schematics": "^0.4.0"

@MichaelWarneke
Copy link
Contributor Author

Done

@MikeRyanDev
Copy link
Member

Merged into next. Thanks!

👏 👏 👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants