-
Notifications
You must be signed in to change notification settings - Fork 25.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
perf: yarn support version incremented up to 1.13.x (the latest version) #28360
Conversation
@jenniferfell as we discussed on #28334, I'm creating a separate PR for yarn package upgrade. Please check. |
6f20322
to
9892cff
Compare
9892cff
to
6075600
Compare
6075600
to
42262f2
Compare
So there's good news and bad news. 👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there. 😕 The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request. Note to project maintainer: This is a terminal state, meaning the |
42262f2
to
6f20322
Compare
CLAs look good, thanks! |
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 for docs-infra changes
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. thanks!
once #28393 lands we should increase the lower bound so that we don't have too many yarn versions floating around.
6f20322
to
074aaf6
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. |
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
yarn support version incremented up to 1.13.x (the latest version). It's package config update which was blocking the development because unavailability of yarn old versions. Also, old version of yarn are not available, and it is stopping local setups. By this PR, the yarn package version will be support the latest yarn, and it will not block the dev to setup angular repo in local machines.
What is the current behavior?
Currently if someone tries to setup the angular repo in local machines with latest version of yarn, the
yarn
command will failed because we are not supporting the yarn recent version. It is difficult to install the old versions of yarn which is not officially supported. So, the development will be getting blocked.Issue Number: N/A
What is the new behavior?
The yarn support version has been upgraded. And so,
yarn
and related setups will not block a dev to build and run the local repo in their machine.Does this PR introduce a breaking change?
Other information