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

Upgrade max version Node.js to the next LTS #45

Merged
merged 2 commits into from
Sep 5, 2024

Conversation

dhoko
Copy link
Contributor

@dhoko dhoko commented Sep 4, 2024

👋

What does it do?

Remove the max version of Node inside engines

Why is it needed?

  • Node.js 22 is the next LTS, soon
  • @strapi/sdk-plugin@5.1.0: The engine "node" is incompatible with this module. Expected version ">=18.0.0 <=20.x.x". Got "22.6.0" 🔥
  • While having a limit to prevent people from using something too old makes sense, the other one is a bit weird. It makes sense to work on a newer version to ensure we can move to the next LTS without issues.

How to test it?

Run yarn

Related issue(s)/PR(s)

N/A

🙏 thx

Copy link

changeset-bot bot commented Sep 4, 2024

🦋 Changeset detected

Latest commit: 358cd04

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@strapi/sdk-plugin Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@innerdvations
Copy link
Collaborator

innerdvations commented Sep 5, 2024

Thanks for the PR! Our policy is generally to keep the upper limit until we've had a chance to test that newer versions work and we officially support it. However, there don't seem to be any breaking changes that would affect this, and it seems to work fine for me too. If you keep the upper limit but bump it to 22 we can add it :)

Signed-off-by: dhoko <dhoko@users.noreply.github.com>
@dhoko dhoko changed the title Remove Max version of Node.js from engines Upgrade max version Node.js to the next LTS Sep 5, 2024
@dhoko
Copy link
Contributor Author

dhoko commented Sep 5, 2024

@innerdvations ✅ upgraded its value to the next LTS

Signed-off-by: Ben Irvin <ben@innerdvations.com>
@innerdvations innerdvations merged commit 345fa5a into strapi:main Sep 5, 2024
2 checks passed
@echoes-hq echoes-hq bot added the echoes: enhancement/chore Chores associated with general upkeep and maintenance (e.g. dependency upgrades, releases) label Sep 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
echoes: enhancement/chore Chores associated with general upkeep and maintenance (e.g. dependency upgrades, releases)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants