Skip to content

ci: separate PR title validation from build jobs - #128

Merged
danielkov merged 2 commits into
mainfrom
ci/separate-pr-title-check
Sep 8, 2026
Merged

ci: separate PR title validation from build jobs#128
danielkov merged 2 commits into
mainfrom
ci/separate-pr-title-check

Conversation

@danielkov

@danielkov danielkov commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Move Conventional Commit title validation into a lightweight, checkout-free workflow. Build CI no longer subscribes to PR edits, so title and description updates do not rerun Rust or macOS builds.

Impact

Repository branch rules must require pr-title separately if title validation should block merging.

Technical details

Separate workflow triggers prevent metadata edits from publishing skipped replacements for required build checks. Title validation runs on every PR edit, including body-only edits, as well as open, reopen, and synchronize events, so an invalid title cannot be bypassed with a description update.

@kit-code-agent kit-code-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

PR edits publish skipped replacements for required build and title checks, allowing failed validation to be bypassed.

Comment thread .github/workflows/ci.yml Outdated

@kit-code-agent kit-code-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found. The changes look good to merge.

@danielkov
danielkov merged commit 5cfce76 into main Sep 8, 2026
6 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.

1 participant