← Back to issue list

docs(HACKING): clarify commit header for merges

View original Github issue

Metadata

Project
starbase
Number
#184
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Docs-only PR by maintainer mr-cal merged into main as commit 234b360, clarifying HACKING.rst guidance on the conventional-commit header used for merge commits (+4/-1 in HACKING.rst).

Resolution: closed resolved

Reason: The pull request itself was merged with an approving review; squash-merge commit 234b360 "docs(HACKING): clarify commit header for merges (#184)" appears in the HACKING.rst history, so the documentation clarification shipped. The single comment was just a note that the CLA check failed because Launchpad was down, not a code concern.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`? ----- Preview here: https://github.com/canonical/starbase/blob/merge-commits/HACKING.rst#chore

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Docs-only PR by maintainer mr-cal merged into main as commit 234b360, clarifying HACKING.rst guidance on the conventional-commit header used for merge commits (+4/-1 in HACKING.rst).
stealth/space-bunny-alpha — closed resolved Merged canonical/starbase#184 as 234b360, clarifying HACKING.rst commit-header guidance for merges.
stealth/space-bunny-alpha — closed resolved Merged #184, updating HACKING.rst to clarify commit headers for merges; merged as commit 234b360 after approval.
qwen/qwen3.6-35b-a3b — — Merged a documentation update to the HACKING file to clarify commit header guidelines for merges. The change was approved by a reviewer and successfully integrated.
qwen3.6-35b-a3b-mtp-q6 — — Merged documentation update clarifying commit header conventions for merge commits in the HACKING guide. A temporary CLA check failure due to Launchpad downtime was resolved, and the maintainer merged the changes.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/starbase#266 (confidence 80%)

    Commit 1327ca6 "docs: update commit headers for merges (#266)" later reworked the same merge-commit header guidance added here.

  • Related To: canonical/starbase#162 (confidence 65%)

    Commit 4b2c78e updated the commit message guidelines that #184 amends.

  • Superseded By: canonical/starbase#365 (confidence 60%)

    Commit 3fe6515 converted repo docs to markdown, replacing HACKING.rst and so removing this text.

Related issues

Issue Project State Summary Similarity
#162 docs(HACKING): update commit message guidelines starbase merged Merged PR #162 into main via commit 4b2c78e, making HACKING commit-message guidance more descriptive and authoritative.
83%
#346 docs(HACKING): add note about revert commits starbase merged Merged PR #346 added guidance to describe revert commits using conventional-commit syntax.
78%
#218 docs(HACKING): document branch names and changelog starbase merged PR #218 was approved and merged, adding 92 lines to HACKING documentation covering branch naming conventions and changelog details.
76%
#55 Add hacking to README.md snapcraft merged Docs-only PR by ted-gould merged via commit 35af890e6, adding the hacking project to README.md (+1/-1). Approved by sergiusens (:+1:) on 2015-10-30.
72%
#2568 docs: consolidate on a simple HACKING.md snapcraft merged Merged PR #2568 via commit d66877fc79, consolidating contributor setup and development guidance into one simple, consistent HACKING.md.
72%
#349 style(HACKING): fix typo starbase merged Merged PR #349 fixed a one-line spelling typo in HACKING; commit daba40f records the change.
72%
#215 docs: add CONTRIBUTING.md imagecraft merged Merged PR #215 in commit b0aa9f2, replacing HACKING.md with a 312-line CONTRIBUTING.md aligned to the project's contribution model.
72%
#266 docs: update commit headers for merges starbase merged Merged documentation update removing superfluous quotes from merge commit headers, changing three lines; recorded in commit 1327ca6.
72%
#791 docs: update CONTRIBUTING.md craft-application merged Docs-only PR #791 merged as commit 60bc3fae, rewriting CONTRIBUTING.md (+278/-319, 3 files) and deleting HACKING.md. Approved by 2 reviewers; only the OSV-scanner check failed.
71%
#297 (draft): docs: reformat README, add HACKING craft-providers closed Superseded by merged PR #296; its README.rst conversion and HACKING.rst documentation landed in commit 1e16ba5.
71%