Skip to content

fix(release): ensure emoji is not repeated in breaking changes summary#33605

Merged
Coly010 merged 2 commits intomasterfrom
release/emoji-in-breaking-changes
Nov 25, 2025
Merged

fix(release): ensure emoji is not repeated in breaking changes summary#33605
Coly010 merged 2 commits intomasterfrom
release/emoji-in-breaking-changes

Conversation

@Coly010
Copy link
Copy Markdown
Contributor

@Coly010 Coly010 commented Nov 25, 2025

Current Behavior

The ⚠️ emoji at the beginning of bc commits is duplicated in the bc section of the changelog.
This is unneeded.

Expected Behavior

Ensure the ⚠️ is not repeated in the bc section of the changelog

@Coly010 Coly010 requested a review from a team as a code owner November 25, 2025 09:55
@Coly010 Coly010 self-assigned this Nov 25, 2025
@Coly010 Coly010 requested a review from Cammisuli November 25, 2025 09:55
@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 25, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
nx-dev Ready Ready Preview Nov 25, 2025 10:53am

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 25, 2025

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit c394154
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/69258923a6b6980008214699
😎 Deploy Preview https://deploy-preview-33605--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Nov 25, 2025

View your CI Pipeline Execution ↗ for commit c394154

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ✅ Succeeded 18m 50s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 2m 46s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 12s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-11-25 11:09:34 UTC

Comment thread packages/nx/release/changelog-renderer/index.ts
Copy link
Copy Markdown
Contributor

@nx-cloud nx-cloud Bot left a comment

Choose a reason for hiding this comment

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

Important

A new CI pipeline execution was requested that may update the conclusion below...

✅ The fix from Nx Cloud was applied

These changes fix the failing e2e tests by updating the snapshots to align with the PR's intentional behavior change. The PR successfully removes the duplicate ⚠️ emoji from breaking change items in the Breaking Changes section of changelogs, and these snapshot updates ensure the e2e tests now expect the corrected output format.

Suggested Fix changes
diff --git a/e2e/release/src/conventional-commits-config.test.ts b/e2e/release/src/conventional-commits-config.test.ts
index 40f183a91f..de71df6881 100644
--- a/e2e/release/src/conventional-commits-config.test.ts
+++ b/e2e/release/src/conventional-commits-config.test.ts
@@ -399,7 +399,7 @@ describe('nx release conventional commits config', () => {
 
       ### ⚠️  Breaking Changes
 
-      - ⚠️  **{project-name}:** this is a breaking change
+      - **{project-name}:** this is a breaking change
     `);
 
     const pkg2Changelog = readFile(`${pkg2}/CHANGELOG.md`);
diff --git a/e2e/release/src/conventional-commits-config.workspaces.test.ts b/e2e/release/src/conventional-commits-config.workspaces.test.ts
index 7365cfced6..e5197c9e01 100644
--- a/e2e/release/src/conventional-commits-config.workspaces.test.ts
+++ b/e2e/release/src/conventional-commits-config.workspaces.test.ts
@@ -406,7 +406,7 @@ describe('nx release conventional commits config', () => {
 
       ### ⚠️  Breaking Changes
 
-      - ⚠️  **{project-name}:** this is a breaking change
+      - **{project-name}:** this is a breaking change
     `);
 
     const pkg2Changelog = readFile(`${pkg2}/CHANGELOG.md`);

Revert fix via Nx Cloud  

View interactive diff ↗

🎓 Learn more about Self-Healing CI on nx.dev

@Coly010 Coly010 merged commit 6c2b3ee into master Nov 25, 2025
20 checks passed
@Coly010 Coly010 deleted the release/emoji-in-breaking-changes branch November 25, 2025 11:14
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 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 Dec 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.

2 participants