Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Node 20 #171

Merged
merged 1 commit into from
Apr 17, 2023
Merged

Add Node 20 #171

merged 1 commit into from
Apr 17, 2023

Conversation

JamieMagee
Copy link
Contributor

Node 20 is set to be released tomorrow1 (2023-04-18). The configuration should be identical to Node 18, as far as I can tell.

Footnotes

  1. https://github.com/nodejs/Release#release-schedule

"compilerOptions": {
"lib": ["es2023"],
"module": "commonjs",
"target": "es2022",
Copy link
Member

Choose a reason for hiding this comment

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

I think we probably want to bump target here also

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ES2022 is the latest target

$ tsc --version
Version 5.0.4
$ tsc --target ES2023
error TS6046: Argument for '--target' option must be: 'es3', 'es5', 'es6', 'es2015', 'es2016', 'es2017', 'es2018', 'es2019', 'es2020', 'es2021', 'es2022', 'esnext'.

Copy link
Member

Choose a reason for hiding this comment

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

Ace 👍🏻

@orta
Copy link
Member

orta commented Apr 17, 2023

Looks good to me aside for the target, nice work

@orta orta merged commit 63a8253 into tsconfig:main Apr 17, 2023
@orta
Copy link
Member

orta commented Apr 17, 2023

Should deploy overnight!

@JamieMagee JamieMagee deleted the node20 branch April 17, 2023 20:51
@hyperupcall hyperupcall mentioned this pull request Nov 5, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants