Skip to content

fix(release): ensure generatorOptions and rootVersionActions options are passed to updateLockfile function#30796

Merged
JamesHenry merged 8 commits intonrwl:masterfrom
thdk:feat/30793/skiplockfileupdate
Apr 25, 2025
Merged

fix(release): ensure generatorOptions and rootVersionActions options are passed to updateLockfile function#30796
JamesHenry merged 8 commits intonrwl:masterfrom
thdk:feat/30793/skiplockfileupdate

Conversation

@thdk
Copy link
Copy Markdown
Contributor

@thdk thdk commented Apr 19, 2025

Current Behavior

nx release version by default will also run package manager commands to update the lockfile after packages have been versioned. Example for npm it will run npm install --lock-file-only. There used to be a flag to disable this behavior but that flag is no longer working.

Expected Behavior

With legacy versioning, one should be able to set release.version.skipUpdateLockfile to true to prevent nx to run any package manager commands to automatically update the lockfile after packages have been versioned.

The new versioning logic introduced in nx 20.8.0 (opt in) should also have an option to disable this behavior.

Related Issue(s)

Fixes #30793

@thdk thdk requested a review from a team as a code owner April 19, 2025 16:35
@thdk thdk requested a review from Coly010 April 19, 2025 16:35
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 19, 2025

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

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Apr 25, 2025 10:50am

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Apr 19, 2025

View your CI Pipeline Execution ↗ for commit 45c4386.

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

☁️ Nx Cloud last updated this comment at 2025-04-25 11:16:21 UTC

@thdk thdk changed the title Ensure generator and rootVersionActions options are pased to updateLockfile function fix(release): ensure generatorOptions and rootVersionActions options are passed to updateLockfile function Apr 19, 2025
Copy link
Copy Markdown
Contributor

@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.

Thank you @thdk!

@JamesHenry JamesHenry enabled auto-merge (squash) April 20, 2025 07:49
@thdk
Copy link
Copy Markdown
Contributor Author

thdk commented Apr 20, 2025

@JamesHenry you're welcome!. Happy to see the release command has received so much attention recently. Looks promising!

CI still fails for this PR though but I don't think that could be caused by these changes. Locally everything is green.

@JamesHenry JamesHenry self-assigned this Apr 22, 2025
@JamesHenry JamesHenry removed the request for review from Coly010 April 22, 2025 08:55
@JamesHenry JamesHenry merged commit 69ea632 into nrwl:master Apr 25, 2025
6 checks passed
@thdk thdk deleted the feat/30793/skiplockfileupdate branch April 25, 2025 13:06
FrozenPandaz pushed a commit that referenced this pull request Apr 25, 2025
…re passed to updateLockfile function (#30796)

(cherry picked from commit 69ea632)
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 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 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Nx release version skipLockFileUpdate no longer working since Nx 20.8

2 participants