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
build: deduplicate node dependencies #23243
build: deduplicate node dependencies #23243
Conversation
18ac62f
to
4f03a48
Compare
"@angular/common": "12.1.2", | ||
"@angular/compiler": "12.1.2", | ||
"@angular/core": "12.1.2", | ||
"@angular/forms": "12.1.2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we have to lock down to a specific version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we'd not lock them there, then this lock file update would have updated to the most recent patch/minor. This would be acceptable but I figured we want to explicitly update Angular versions together with it being reflected in the package.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
4f03a48
to
eabd624
Compare
Deduplicates some node dependencies by cleaning up the Yarn lock file so that dependencies can be deduplicated and merged together (to a version that satisfies multiple ranges).
eabd624
to
c15af0c
Compare
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
See individual commits.