-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
fix(js): do not overwrite supported typescript version #17350
fix(js): do not overwrite supported typescript version #17350
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
0257c8e
to
5528dc8
Compare
As this popped up for Angular, would you mind also fixing this for other packages? Maybe a more general approach would be then necessary. "ng-packagr" is simply overwritten in the angular library generator and does not care about the currently installed version either. https://github.com/nrwl/nx/blob/master/packages/angular/src/generators/library/library.ts#L100 I know it does not really relate to this MR. If you prefer I can create a new issue instead. EDIT: Thanks for fixing it :) |
5528dc8
to
87a0922
Compare
87a0922
to
1f437c1
Compare
(cherry picked from commit c68b4bf)
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Current Behavior
The
@nx/js:init
generator always sets thetypescript
package version to the latest supported version. This causes issues where an older supported version is needed in the workspace (e.g. when using an older supported version of Angular).Expected Behavior
The
@nx/js:init
generator should not overwrite thetypescript
package version if it's a supported version.Related Issue(s)
Fixes #17344