-
Notifications
You must be signed in to change notification settings - Fork 6.8k
build: update node and yarn to latest stable versions #17474
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
Conversation
4e38a5e
to
12f06d5
Compare
For reference: a few goldens had to be updated because the method overloads are now sorted based on their declaration order. This previously was not the case because |
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
e730954
to
d8d6785
Compare
@josephperrott Feedback addressed. Please have another look |
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
Some CI jobs currently fail because db283f1 removed Node v11 from the supported node versions. Some of CI jobs use Node V11 though, so they currently fail. e.g. https://circleci.com/gh/angular/components/89692. In order to fix this, we ensure that we use consistent versions. For Node, we use the current active LTS version (v12). And for Yarn, we use the latest stable yarn version.
d8d6785
to
f0e5608
Compare
How is requiring NodeJS 12 working with relation to the fact that Gulp 3 doesn't work with it? We're still on |
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. |
Some CI jobs currently fail because db283f1 removed
Node v11 from the supported node versions. Some of CI jobs use Node v11 though, so they
currently fail. e.g. https://circleci.com/gh/angular/components/89692.
In order to fix this, we ensure that we use consistent versions. For Node, we use the current
active LTS version (v12). And for Yarn, we use the latest stable yarn version.