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

feat: increase minimum node version to v18 #106

Merged
merged 4 commits into from
Sep 7, 2023

Conversation

meeroslav
Copy link
Collaborator

@meeroslav meeroslav commented Sep 6, 2023

No description provided.

@meeroslav meeroslav self-assigned this Sep 6, 2023
@meeroslav meeroslav force-pushed the switch-to-yarn branch 2 times, most recently from 636da76 to ea2b97a Compare September 6, 2023 10:49
@meeroslav meeroslav changed the title feat: increase minimum node version to v16 feat: increase minimum node version to v18 Sep 6, 2023
@meeroslav meeroslav force-pushed the switch-to-yarn branch 3 times, most recently from 34a925c to 3e39ef6 Compare September 6, 2023 12:55
@meeroslav meeroslav changed the title feat: increase minimum node version to v18 feat: increase minimum node version to v20 Sep 6, 2023
@meeroslav meeroslav removed the request for review from StalkAltan September 6, 2023 12:56
JamesHenry

This comment was marked as outdated.

package.json Outdated Show resolved Hide resolved
@JamesHenry JamesHenry changed the title feat: increase minimum node version to v20 feat: increase minimum node version to v18 Sep 6, 2023
@JamesHenry
Copy link
Collaborator

Summary of what we discussed:

Because this is a composite action we do not actually get to control the version of node that the action runs in, it will be inherited from the user's own pipeline.

This means that all we can do on our side is express our desired minimum supported version and build and test using that version to ensure the best chance of compatibility.

Given Node 18 will still be supported for another year, and some popular tools out there still do not explicitly support Node 20 it is safer for us to stick with 18 for now.

@meeroslav meeroslav merged commit 0712662 into nrwl:main Sep 7, 2023
6 checks passed
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.

2 participants