Skip to content

test(version): add e2e tests for breaking change conventions#4312

Merged
JamesHenry merged 1 commit intolerna:mainfrom
AI-JamesHenry-Org:2668
Mar 13, 2026
Merged

test(version): add e2e tests for breaking change conventions#4312
JamesHenry merged 1 commit intolerna:mainfrom
AI-JamesHenry-Org:2668

Conversation

@AI-JamesHenry
Copy link
Copy Markdown
Collaborator

@AI-JamesHenry AI-JamesHenry commented Mar 13, 2026

Note

🤖 This PR was created by @AI-JamesHenry, an AI assistant account guided and overseen by @JamesHenry.

Summary

  • Adds e2e tests documenting lerna's default behavior with the three conventional commit breaking change notations using the default angular preset
  • Covers BREAKING CHANGE: footer (works — triggers major bump), ! notation only (does not trigger major bump with angular preset), and both combined (works via footer)
  • Captures the historical behavior discussed in Bug: Conventional commits do not respect '!' or "BREAKING CHANGE" #2668

Test plan

  • nx e2e e2e-version --testPathPattern="breaking-changes" passes locally (all 3 tests green, snapshots stable across runs)
  • CI passes

Add e2e tests documenting lerna's default behavior with the three
conventional commit breaking change notations (BREAKING CHANGE footer,
! notation, and both combined) using the default angular preset.
@nx-cloud
Copy link
Copy Markdown

nx-cloud bot commented Mar 13, 2026

View your CI Pipeline Execution ↗ for commit 224ef11

Command Status Duration Result
nx run-many --t e2e --parallel=1 ✅ Succeeded 12m 9s View ↗
nx run-many -t lint --parallel=3 ✅ Succeeded 5s View ↗
nx-cloud record -- npx nx format:check ✅ Succeeded 2s View ↗
nx run-many -t test --parallel=3 --ci --maxWork... ✅ Succeeded <1s View ↗
nx run integration:integration --ci --maxWorkers=2 ✅ Succeeded <1s View ↗
nx run-many -t build --parallel=3 ✅ Succeeded <1s View ↗
nx run-many -t build ✅ Succeeded <1s View ↗
nx run-many -t test --parallel=3 --ci --maxWork... ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-03-13 14:05:58 UTC

@JamesHenry JamesHenry marked this pull request as ready for review March 13, 2026 14:39
@JamesHenry JamesHenry merged commit 77f5a25 into lerna:main Mar 13, 2026
15 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