Skip to content

Conversation

ndcunningham
Copy link
Contributor

@ndcunningham ndcunningham commented Apr 22, 2025

BREAKING CHANGE: We are updating our minimum node version to 20.19.0 which drops support for all prior versions.

@ndcunningham ndcunningham self-assigned this Apr 22, 2025
@ndcunningham ndcunningham requested a review from a team as a code owner April 22, 2025 19:20
@ndcunningham ndcunningham requested a review from AgentEnder April 22, 2025 19:20
Copy link

vercel bot commented Apr 22, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview May 1, 2025 5:59am
Copy link
Contributor

nx-cloud bot commented Apr 22, 2025

View your CI Pipeline Execution ↗ for commit 7cc58a7.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-ci ✅ Succeeded 11m 31s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 16s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 1s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗
nx documentation ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-05-01 06:13:24 UTC

Copy link
Collaborator

@JamesHenry JamesHenry left a comment

Choose a reason for hiding this comment

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

See slack thread

@ndcunningham ndcunningham force-pushed the feat/upgrade-to-node-20 branch from 5a0dd8e to a84176c Compare April 22, 2025 20:15
@ndcunningham ndcunningham requested review from a team as code owners April 22, 2025 20:15
@ndcunningham ndcunningham changed the title feat(core): Update base Node version to 20 Apr 22, 2025
@ndcunningham ndcunningham requested a review from JamesHenry April 22, 2025 20:20
@ndcunningham ndcunningham force-pushed the feat/upgrade-to-node-20 branch from a84176c to b861e7d Compare April 22, 2025 20:45
Copy link
Contributor

🐳 We have a release for that!

This PR has a release associated with it. You can try it out using this command:

npx create-nx-workspace@0.0.0-pr-30818-a84176c my-workspace

Or just copy this version and use it in your own command:

0.0.0-pr-30818-a84176c
Release details 📑
Published version 0.0.0-pr-30818-a84176c
Triggered by @ndcunningham
Branch feat/upgrade-to-node-20
Commit a84176c
Workflow run 14604007199

To request a new release for this pull request, mention someone from the Nx team or the @nrwl/nx-pipelines-reviewers.

Copy link
Contributor

🐳 We have a release for that!

This PR has a release associated with it. You can try it out using this command:

npx create-nx-workspace@0.0.0-pr-30818-b861e7d my-workspace

Or just copy this version and use it in your own command:

0.0.0-pr-30818-b861e7d
Release details 📑
Published version 0.0.0-pr-30818-b861e7d
Triggered by @ndcunningham
Branch feat/upgrade-to-node-20
Commit b861e7d
Workflow run 14605127935

To request a new release for this pull request, mention someone from the Nx team or the @nrwl/nx-pipelines-reviewers.

@beaussan
Copy link
Contributor

Hey there 👋

I suspect the node 20.19 was picked for the require(esm) support

If that's the case, you may want to support only node 22.12 and up instead of 22.x

Since it's not enabled under 22.12 by default, you would have node version in your support range that don't support require(esm)

See release notes for 22.12 https://nodejs.org/en/blog/release/v22.12.0

@jaysoo jaysoo force-pushed the feat/upgrade-to-node-20 branch from 19400e3 to ca9f2e0 Compare May 1, 2025 02:13
@jaysoo jaysoo requested review from a team and Coly010 as code owners May 1, 2025 03:58
@jaysoo jaysoo force-pushed the feat/upgrade-to-node-20 branch from 25c818c to 55787f2 Compare May 1, 2025 04:23
@jaysoo jaysoo force-pushed the feat/upgrade-to-node-20 branch from 9904ce1 to d1b3901 Compare May 1, 2025 05:10
@jaysoo jaysoo force-pushed the feat/upgrade-to-node-20 branch from b56ca3f to 7cc58a7 Compare May 1, 2025 05:56
@jaysoo jaysoo merged commit 28ddb8e into master May 1, 2025
6 checks passed
@jaysoo jaysoo deleted the feat/upgrade-to-node-20 branch May 1, 2025 06:13
Copy link
Contributor

github-actions bot commented May 7, 2025

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

PR status: breaking-changes PR contains breaking changes scope: core core nx functionality

4 participants