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

fix: drop node v14 #1399

Merged
merged 1 commit into from May 11, 2023
Merged

fix: drop node v14 #1399

merged 1 commit into from May 11, 2023

Conversation

NozomuIkuta
Copy link
Contributor

No related issues.

Node versions specified in the codebase and docs are inconsistent.
This PR is to unify Node versions to 16 for 2 reasons:

  • only Node 16 is tested in CI (i.e. .github/workflows/ci.yml)
  • the associated commit to .github/workflows/ci.yml is newer than other places (e.g. commit to package.json)

⚠️ This is my very first PR for this repository, so I might be missing something.

@kazupon kazupon changed the title fix: fix inconsistent node versions fix: drop node v14 May 11, 2023
@kazupon kazupon added the Type: Improvement Includes backwards-compatible fixes label May 11, 2023
@kazupon
Copy link
Member

kazupon commented May 11, 2023

Thank you for your contribution!

node v14 has already passed LTS, so I've changed the title name of this PR :)
https://github.com/nodejs/release#release-schedule

@kazupon kazupon merged commit 4348a61 into intlify:master May 11, 2023
23 checks passed
@NozomuIkuta NozomuIkuta deleted the fix/engine branch May 11, 2023 03:36
@stanleyxu2005
Copy link

stanleyxu2005 commented Oct 3, 2023

Hello contributors.

I'd like to know, if the new version 9.3 (or higher) relies on any Node 16+ specific features?
As you stated above

only Node 16 is tested in CI (i.e. .github/workflows/ci.yml)

It's fine to use Node 16 in CI, but it would be harmless for users that still running a Node 14 with their own risk?

I'm embarrassing to say but I'm still not able to upgrade all my old projects to Node 16+ due to breaking changes of Node, e.g. require('fs') must be require('node:fs')

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Improvement Includes backwards-compatible fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants