Skip to content
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

Closed
wants to merge 4 commits into from

Conversation

orendain
Copy link

@orendain orendain commented Feb 7, 2024

Resolves #182.

Modifies the action to run on Node 20 instead on Node 16.

@CLAassistant
Copy link

CLAassistant commented Feb 7, 2024

CLA assistant check
All committers have signed the CLA.

@orendain
Copy link
Author

@doriable / @rubensf / @smallsamantha , may we get someone to take a look and review?

emcfarlane
emcfarlane previously approved these changes Feb 16, 2024
Copy link
Contributor

@emcfarlane emcfarlane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! Could you update the package.json here too:

@emcfarlane emcfarlane dismissed their stale review February 16, 2024 20:50

Requires updates

@orendain
Copy link
Author

Thanks for the PR! Could you update the package.json here too:

Thanks! Fixed.

emcfarlane added a commit that referenced this pull request Feb 20, 2024
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
@emcfarlane
Copy link
Contributor

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.

@emcfarlane emcfarlane closed this Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bump the action to run on Node 20 instead of deprecated Node 16.
3 participants