← Back to issue list

feat: switch per-commit conventional commit check to PR title

View original Github issue

Metadata

Project
starflow
Number
#109
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

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.
79%
#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.
74%
#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.
70%