Skip to content

Bump node from 16 to 20#1104

Merged
pavera merged 1 commit into
mainfrom
mctofu/node20
May 9, 2024
Merged

Bump node from 16 to 20#1104
pavera merged 1 commit into
mainfrom
mctofu/node20

Conversation

@mctofu

@mctofu mctofu commented Nov 6, 2023

Copy link
Copy Markdown
Contributor

Updates the action to run with node 20.

This works, but before merging we need a strategy for supporting updates to older GHES versions running node 16.

@martincostello

Copy link
Copy Markdown

Just bumping this as I've switched to the new support for actions to run dependabot and spotted the warning about Node.js v16.

@pavera pavera marked this pull request as ready for review May 8, 2024 15:48
@pavera pavera requested a review from a team as a code owner May 8, 2024 15:48
@pavera pavera merged commit 4124341 into main May 9, 2024
@pavera pavera deleted the mctofu/node20 branch May 9, 2024 17:01
@Nishnha Nishnha mentioned this pull request May 31, 2024
jeffwidman added a commit that referenced this pull request Mar 21, 2025
`es6` is legacy because GitHub actions used to use `node12`, which didn't support some of the newer versions.

But GitHub Actions is now on `node20`, which supports `es2022`, so it's safe to let Typescript compile using this much newer syntax.

We bumped this particular action to node20 here:
* #1104

I'm starting to see Dependabot PR's trying to bump dependencies that are
erroring because they can't use newer javascript features, so bumping this
unblocks them.
jeffwidman added a commit that referenced this pull request Mar 21, 2025
`es6` is legacy because GitHub actions used to use `node12`, which didn't support some of the newer versions.

But GitHub Actions is now on `node20`, which supports `es2022`, so it's safe to let Typescript compile using this much newer syntax.

We bumped this particular action to node20 here:
* #1104

I'm starting to see Dependabot PR's trying to bump dependencies that are
erroring because they can't use newer javascript features, so bumping this
unblocks them.
@thebaptiste

Copy link
Copy Markdown

This action should be updated to use Node 24 now (Node 20 is deprecated)

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.

6 participants