Skip to content

chore(release): explicitly push git commit and tags after nx release#445

Merged
justinorringer merged 1 commit intoRedHatInsights:mainfrom
charlesmulder:RHCLOUD-45816
Mar 12, 2026
Merged

chore(release): explicitly push git commit and tags after nx release#445
justinorringer merged 1 commit intoRedHatInsights:mainfrom
charlesmulder:RHCLOUD-45816

Conversation

@charlesmulder
Copy link
Contributor

@charlesmulder charlesmulder commented Mar 12, 2026

Problem

After switching to the unified npx nx release command, packages were being published to npm but the chore(release): publish commit was never pushed back to the repo. This prevented nacho-bot from creating PRs to update version field in package.json.

Root Cause

The unified nx release command ignores release.git.push: true in nx.json. It only pushes automatically when createRelease (GitHub/GitLab releases) is enabled. Since we use createReleaseInGitHub: false, no push ever happened — the commit existed only in the ephemeral CI environment before being discarded.

This affects nx 21.5.3 (javascript-clients) and nx 22.3.3 (frontend-components) identically.

Fix

Add an explicit git push --follow-tags step after npx nx release --yes.

Also manually syncs rbac (6.0.1 → 7.0.0) and host-inventory (4.1.7 → 5.0.0) package.json versions to match what was already published to npm.

@charlesmulder charlesmulder changed the title fix(release): explicitly push git commit and tags after nx release chore(release): explicitly push git commit and tags after nx release Mar 12, 2026
Copy link
Contributor

@justinorringer justinorringer left a comment

Choose a reason for hiding this comment

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

LGTM!

@justinorringer justinorringer merged commit 16efc62 into RedHatInsights:main Mar 12, 2026
9 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