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

Update docs node version requirement to reflect current redwood reqs #5009

Merged
merged 3 commits into from
Apr 3, 2022

Conversation

dkmooers
Copy link
Contributor

@dkmooers dkmooers commented Apr 3, 2022

Wasn't sure whether to keep <= 16.x format from current docs, or switch to < 17.0.0 from current redwood CLI error message, but opted for the latter

@netlify
Copy link

netlify bot commented Apr 3, 2022

Deploy Preview for redwoodjs-docs ready!

Name Link
🔨 Latest commit e4a540c
🔍 Latest deploy log https://app.netlify.com/sites/redwoodjs-docs/deploys/6249df5c2603a30008305ebd
😎 Deploy Preview https://deploy-preview-5009--redwoodjs-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@dkmooers
Copy link
Contributor Author

dkmooers commented Apr 3, 2022

BTW this is my first commit to an open source project, sorry if I borked something!

@dkmooers dkmooers changed the title Update docs node version requirement to reflect current redwood reqs Update docs node version requirement to reflect current redwood reqs release:chore Apr 3, 2022
@dkmooers dkmooers changed the title Update docs node version requirement to reflect current redwood reqs release:chore Update docs node version requirement to reflect current redwood reqs Apr 3, 2022
@dkmooers dkmooers closed this Apr 3, 2022
@dkmooers dkmooers reopened this Apr 3, 2022
@dkmooers
Copy link
Contributor Author

dkmooers commented Apr 3, 2022

Am I supposed to add a release:chore label to this PR somehow? Not sure if I have privileges to do that, can't figure out how

@thedavidprice
Copy link
Contributor

Welcome to Redwood! Glad to have you as a contributor.

We will take care of the label.

Do see my comment about bumping the version engine. Node.js does things in a unique way.

@thedavidprice thedavidprice added the release:chore This PR is a chore (means nothing for users) label Apr 3, 2022
@@ -30,7 +30,7 @@ If you have an existing site created with a prior version, you'll need to upgrad

During installation, RedwoodJS checks if your system meets version requirements for Node and Yarn:

- node: ">=14.17 <=16.x"
- node: ">=14.19.0 <17.0.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Node versions are a peculiarity. In general, odd numbered versions are short-lived, development releases that are never long-term supported. Therefore, we do not support them. It's worth looking up the Node.js release schedule if you're never seen it before.

But we did bump the minimum to 14.19 for security and stability. Also, we don't include the patch semver because it may increase and that's desired to include.

So min should be 14.19 and max should stay as is.

@dkmooers
Copy link
Contributor Author

dkmooers commented Apr 3, 2022

OK great, updated min version to 14.19 and left max as is.

@thedavidprice
Copy link
Contributor

Thank you! Merging now

@thedavidprice thedavidprice merged commit 52e96a1 into redwoodjs:main Apr 3, 2022
@jtoar jtoar added this to the next-release milestone Apr 3, 2022
@dkmooers
Copy link
Contributor Author

dkmooers commented Apr 3, 2022

Yeah for sure, thanks!

@dkmooers dkmooers deleted the docs-updates branch April 3, 2022 17:55
@thedavidprice thedavidprice modified the milestones: next-release, v1.1.0 Apr 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:chore This PR is a chore (means nothing for users)
Projects
No open projects
Status: Archived
Development

Successfully merging this pull request may close these issues.

3 participants