Skip to content

ci: add breaking change guard to PR workflow#206

Merged
emilyoram merged 2 commits into
mainfrom
feat/breaking-change-guard
May 18, 2026
Merged

ci: add breaking change guard to PR workflow#206
emilyoram merged 2 commits into
mainfrom
feat/breaking-change-guard

Conversation

@emilyoram
Copy link
Copy Markdown
Contributor

Adds the shared sdk-breaking-change-guard reusable workflow from sdk-infra to PR CI. This fails the build when PR commits contain BREAKING CHANGE: notes that would trigger an unintended major version bump via semantic-release.

Changes

  • .github/workflows/ci.yml: Added breaking-change-guard job (runs on PRs only)
  • AGENTS.md: Documented breaking change commit conventions and the bypass label

Bypass

Add the breaking-change-approved label to skip the guard for intentional breaking changes.

Closes #205

emilyoram added 2 commits May 15, 2026 14:13
Replace \git add -A\ with an explicit allowlist of expected generated
paths (Generated/, external-spec/bundled/, spec-snapshots/) to prevent
any unexpected files from being auto-committed to protected branches.

Pin all devDependencies to exact versions (remove ^ ranges) so that
\
pm ci\ is fully deterministic and a compromised semver-compatible
release of camunda-schema-bundler or other deps cannot be silently
picked up. Dependabot/Renovate will handle version bumps via PR review.

Addresses camunda/security-testing-findings#66.
@emilyoram emilyoram merged commit 596d166 into main May 18, 2026
15 of 16 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.

Add breaking change commit detection to PR CI

2 participants