schematics icon indicating copy to clipboard operation
schematics copied to clipboard

@ngxs/schematics should be added to devDependencies

Open xmlking opened this issue 5 years ago • 10 comments

I'm submitting a...


[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report  
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request => https://github.com/ngxs/store/blob/master/CONTRIBUTING.md
[ ] Other... Please describe:

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


Libs:
- @angular/core version: X.Y.Z
- @ngxs/store version: X.Y.Z


Browser:
- [ ] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX
 
For Tooling issues:
- Node version: XX  
- Platform:  

Others:

xmlking avatar Dec 01 '18 18:12 xmlking

Should it reference itself in its dev dependencies?

markwhitfeld avatar Dec 01 '18 18:12 markwhitfeld

i mean something like this https://github.com/nrwl/nx/blob/master/packages/schematics/src/collection/ng-add/index.ts#L97

xmlking avatar Dec 01 '18 19:12 xmlking

may be we can change the type here https://github.com/ngxs/schematics/blob/master/src/utils/common/lib.config.ts#L44

xmlking avatar Dec 01 '18 19:12 xmlking

Yes that makes sense

markwhitfeld avatar Dec 02 '18 07:12 markwhitfeld

@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 save property by default. In our case, we need to improve it and make flag that will understand where package should added.

kyusupov33 avatar Dec 20 '18 06:12 kyusupov33

https://github.com/angular/angular-cli/issues/13237

kyusupov33 avatar Dec 20 '18 07:12 kyusupov33

@kyusupov33 Thanks for the heads up, I also commented there.

eranshmil avatar Dec 20 '18 08:12 eranshmil

@kyusupov33 the author of this package, it is strange that he was expelled

splincode avatar Dec 20 '18 08:12 splincode

@splincode no idea how that happened. Sorry @kyusupov33, I have added you back as a collaborator.

markwhitfeld avatar Feb 13 '19 08:02 markwhitfeld

@markwhitfeld thanks :)

kyusupov33 avatar Feb 13 '19 08:02 kyusupov33