-
Notifications
You must be signed in to change notification settings - Fork 23
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
Upgrade to Node 16 #418
Upgrade to Node 16 #418
Conversation
3e0aa53
to
ed98e66
Compare
Oh wow, 8 hours ago. I had not realized that. Good job, I was going to ping someone for review but we'll give them time to breathe! |
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.
Approving, but I have not reviewed the package-lock.json carefully.
There is also coursier/cache-action that needs upgrading but we'll open a PR when coursier/cache-action#418 is merged
ed98e66
to
d586ec9
Compare
can you approve the workflow @alexarchambault just so we have a green pipeline and see if this needs work? |
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.
Thanks @BalmungSan!
Fixes #410