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

feat: update to use node20 #61

Merged
merged 5 commits into from
Dec 30, 2023
Merged

feat: update to use node20 #61

merged 5 commits into from
Dec 30, 2023

Conversation

chenrui333
Copy link
Contributor

This is consistent with all the recent github action updates. (mostly in actions repos, like actions/checkout, actions/setup-node, etc.)


Node12 was deleted from runner. Node20 was added to Actions Runner on v2.308.0.
Node16 has en end of life on 11 Sep 2023.

This PR updates the default runtime to node20, rather then node16

relates to actions/runner#2732

@chenrui333
Copy link
Contributor Author

cc @goto-bus-stop

@chenrui333
Copy link
Contributor Author

looks like there is some issue with testing against zig 0.11.0, https://github.com/chenrui333/setup-zig/actions/runs/7296724152/job/19884920370

@goto-bus-stop
Copy link
Owner

thanks for the PR! zig 0.11 changed the build system so i've pulled that out into #62.

@goto-bus-stop goto-bus-stop merged commit 426edce into goto-bus-stop:default Dec 30, 2023
@chenrui333
Copy link
Contributor Author

@goto-bus-stop also the setup action should be bumped to v3 per this upgrade. :)

@goto-bus-stop
Copy link
Owner

goto-bus-stop commented Jan 3, 2024

imo it's OK to treat this bump as non-breaking since the environment it runs in is known to be compatible (aside from self-hosted runners, maybe?) and the cost of a major version bump on actions is very high as most projects do not have dependabot configured for actions.

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.

None yet

2 participants