-
Notifications
You must be signed in to change notification settings - Fork 16
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
@ngxs/schematics should be added to devDependencies #16
Comments
Should it reference itself in its dev dependencies? |
i mean something like this |
may be we can change the |
Yes that makes sense |
@eranshmil @markwhitfeld your solution doesn't solve that problem, because we need to send PR in angular-cli with change on this file: https://github.com/angular/angular-cli/blob/master/packages/angular/cli/tasks/npm-install.ts As you can see, they set true in |
@kyusupov33 Thanks for the heads up, I also commented there. |
@kyusupov33 the author of this package, it is strange that he was expelled |
@splincode no idea how that happened. |
@markwhitfeld thanks :) |
I'm submitting a...
Current behavior
@ngxs/schematics added to Dependencies
Expected behavior
@ngxs/schematics should be added to devDependencies
Minimal reproduction of the problem with instructions
What is the motivation / use case for changing the behavior?
Environment
The text was updated successfully, but these errors were encountered: