feat: switch per-commit conventional commit check to PR title
Metadata
Current evaluation
Merged #109 as 7a85232, replacing per-commit validation with PR-title validation; follow-up #112 (3a1c176) limited the check to PR events.
Resolution: closed resolved
Reason: PR #109 was merged as commit 7a85232. Its merge-commit failure was addressed by follow-up PR #112, merged as 3a1c176.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This switches out the conventional commits check so instead of checking the titles of each actual commit, we're just checking the PR title. Almost all PRs get squashed, so this makes life easier.
CRAFT-4977
Example failure: https://github.com/canonical/starflow/actions/runs/21528045507/job/62036683150?pr=109
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #109 as 7a85232, replacing per-commit validation with PR-title validation; follow-up #112 (3a1c176) limited the check to PR events. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #109 (7a85232), replacing per-commit Conventional Commit checks with a PR-title check; follow-up #112 (3a1c176) stopped merge commits from triggering it. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a change replacing per-commit conventional commit validation with PR title checking to streamline squashed PRs. Approved by two reviewers, the update passed CI despite an initial false positive on the merge commit. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Replaced per-commit conventional commit validation with PR title checking to streamline squashed workflows. The merge commit itself subsequently failed the newly implemented check. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/starflow#112
(confidence 100%)
Restricts the new Conventional Commits check to pull-request events, preventing merge commits from triggering it.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #112 fix: only run conventional commits check on PRs | starflow | merged | Merged PR #112 as commit 3a1c176, limiting the Conventional Commits title check to pull-request events. | |
| #482 docs: mandate conventional commits in PR titles | starbase | merged | Merged PR #482 as commit 602d1d0, requiring Conventional Commit-style PR titles and explaining which commit title to use. | |
| #1065 docs: Update CONTRIBUTING.md for PR titles | rockcraft | merged | Merged PR #1065 in commit 9df5b23c, updating CONTRIBUTING.md to clarify that Conventional Commits style applies to pull request titles. |