← 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 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.

Suggested action: —

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
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 issues

Issue Project State Summary Similarity
#482 docs: mandate conventional commits in PR titles starbase merged Merged an update to CONTRIBUTING.md mandating conventional commits in PR titles. Approved by two reviewers, passed all CI checks, and applied with minimal documentation changes.
79%
#112 fix: only run conventional commits check on PRs starflow merged Merged a fix that restricts the conventional commits check to pull requests only. Approved by two reviewers, passed all CI checks, and applied a single-line change to one file.
77%
#1065 docs: Update CONTRIBUTING.md for PR titles rockcraft merged Merged documentation update to CONTRIBUTING.md explicitly requiring Conventional Commits style for PR titles. Approved by reviewers, passed CI checks, and integrated. Maintainers discussed potential future enforcement.
76%