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

Use Node 20 #1

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

AndrewADev
Copy link

@AndrewADev AndrewADev commented Feb 9, 2024

Changes to switch to Node 20, as 16 is no longer supported and GitHub is deprecating Node 16 actions.

Given the deprecation, these changes make a hard cut, and drop support for 16.

Also included are updates to the actions used internally, so that they are on versions using Node 20 as well

image
(Warnings when running the workflow with the previous versions)

Note that this upgrade should help unblock a fix for a similar issue for the action-docker-layer-caching action, which uses this action in the workflows.

CI checks passing on my fork:
image

Update runtime to Node v20, including types package, as well as Node version used in Workflow.
We were seeing an issue running the main workflow after the upgrade to Node v20.

The issue seems to match this one here: cevek/ttypescript#150
This fixes warnings that show up when running the workflow.
Update the examples in the README to use Node v20, as well as to use more recent versions (via tag) of the action itself.
@AndrewADev
Copy link
Author

Hey @jpribyl , do you think you'd be able to take a look at this?

@jpribyl
Copy link
Owner

jpribyl commented Feb 15, 2024

Hey @jpribyl , do you think you'd be able to take a look at this?

Totally! yeah I'll try to carve out some time this week for it. Thanks for the PR!

@mendhak
Copy link

mendhak commented Feb 26, 2024

I've also got a PR in action-docker-layer-caching, but that depends on this PR. Specifically I think the using: node20 in action.yml needs this update.

jpribyl/action-docker-layer-caching#56

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.

3 participants