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

fix(release): ensure git push option is always applied #29596

Merged
merged 3 commits into from
Jan 14, 2025

Conversation

thdk
Copy link
Contributor

@thdk thdk commented Jan 11, 2025

Current Behavior

Running nx release --git-push doesn't push changes to the remote.

Running nx release --git-push --projects my-project does push changes to the remote.

Expected Behavior

nx release commands should always push commits and tags whenever git push (either in nx.json or from command arguments) is set.

Related Issue(s)

Fixes #29595

@thdk thdk requested a review from a team as a code owner January 11, 2025 22:53
@thdk thdk requested a review from FrozenPandaz January 11, 2025 22:53
Copy link

vercel bot commented Jan 11, 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 Jan 11, 2025 10:55pm

@thdk thdk changed the title Fix/29595/nx release git push fix fix(release): git push option is not always working Jan 11, 2025
Copy link

nx-cloud bot commented Jan 11, 2025

View your CI Pipeline Execution ↗ for commit d86822c.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 34m 32s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 1m View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 1s View ↗
nx-cloud record -- nx format:check --base=f57d8... ✅ Succeeded 20s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 19s View ↗
nx documentation --no-dte ✅ Succeeded 1m 3s View ↗

☁️ Nx Cloud last updated this comment at 2025-01-11 23:36:54 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.

Thank you @thdk!

@JamesHenry JamesHenry merged commit bde8a21 into nrwl:master Jan 14, 2025
6 checks passed
@JamesHenry JamesHenry changed the title fix(release): git push option is not always working fix(release): ensure git push option is always applied Jan 14, 2025
@JamesHenry
Copy link
Collaborator

Note, I reworded your PR pre-merge to read more like a resolution rather than an issue description

FrozenPandaz pushed a commit that referenced this pull request Jan 15, 2025
Copy link

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 Jan 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants