-
Notifications
You must be signed in to change notification settings - Fork 24
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
Update the version of Node the action runs on from Node 16 to Node 20 #183
Conversation
@doriable / @rubensf / @smallsamantha , may we get someone to take a look and review? |
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.
…-action into eo/update-action-node
Thanks! Fixed. |
Upgrade dependencies to latest and node runtime to v20. Following dependencies were upgraded with major version changes: - typescript and eslint: not an issue because the build and lint steps still pass - prettier: new formatting does not affect code - @actions/github: the only breaking change appears to be dropped support for Node 16 All code changes are related to formatting and LICENSE years. Fixes: - #183 - #173 - #172 - #159 - #149 - #144 - #141 - #140 - #137
Hey @orendain, we've pulled in and fixed as part of #185 which includes library version updates as well as the node runtime bump to v20. A new release of the actions package will be created when the buf cli is released. For now you can reference the commit hash to pull in the node changes. Thanks again for putting up the issue and the PR. |
Resolves #182.
Modifies the action to run on Node 20 instead on Node 16.