Skip to content

Conversation

@jeffwidman
Copy link
Member

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:

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.

`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.
Copilot AI review requested due to automatic review settings March 21, 2025 19:37
@jeffwidman jeffwidman requested a review from a team as a code owner March 21, 2025 19:37
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

@jeffwidman jeffwidman merged commit 5bb867f into main Mar 21, 2025
9 checks passed
@jeffwidman jeffwidman deleted the bump-target-to-es2022 branch March 21, 2025 19:39
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