-
-
Notifications
You must be signed in to change notification settings - Fork 257
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
chore: Upgrade to node 16 #8215
Conversation
Due to how we detect changes in CI, this won't run all the tests though :/ Is there a way to forcefully run? |
Yes you need to label the PR with run-all-tests then commit sth |
Also, want to try |
Breaking changes introduced by npm We will need to change our scripts a little. |
c385246
to
48f46e3
Compare
Assigning to @clemsos for the smart contract tests issues... |
Team work is dream work! |
Annoyingly a bunch of styling things were added. It looks like my vscode prettyfying config has changed? Any hint @searchableguy on how to fix this? |
Also that fixes #7510 |
e70415d
to
3bb8fad
Compare
3bb8fad
to
83c2c10
Compare
Ok it looks like we have a green build! |
Description
Upgrade node version in our dockerfile to node 16. This should apply to all services using the file.
Let's see if it breaks anything.
Issues
Fixes #
Refs #
Checklist:
Release Note Draft Snippet