docs(HACKING): clarify commit header for merges
Metadata
Current evaluation
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.
Suggested action: —
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 |
|---|---|---|---|---|
| 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 issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #346 docs(HACKING): add note about revert commits | starbase | merged | Merged documentation update to the HACKING guide. Added guidance on formatting revert commits using conventional commit standards. Approved by one reviewer, passed CI checks, and merged with a single line addition. | |
| #162 docs(HACKING): update commit message guidelines | starbase | merged | Merged updates to commit message guidelines in HACKING.rst for greater clarity. Approved by three reviewers, passed CI with 100% coverage, and modified three files. Minor commit type discussion did not block integration. | |
| #218 docs(HACKING): document branch names and changelog | starbase | merged | Merged documentation updates to the HACKING file covering branch naming conventions and changelog procedures. Approved by two reviewers, the change adds 92 lines to a single file. | |
| #225 docs(HACKING): add details on changelog convention | starbase | merged | Merged documentation update adding changelog convention details to the HACKING guide. Approved by two reviewers, passed CI checks, and subsequently upstreamed to the canonical/starbase repository. | |
| #226 docs: update hacking with explanation for exclamation point | starbase | merged | Merged documentation update to the hacking guidelines. Added an explanation for using exclamation points in commit message headers. Approved by two reviewers with a +13/-3 line change across one file. | |
| #266 docs: update commit headers for merges | starbase | merged | Merged documentation update removing superfluous quotes from merge commit headers. Approved by two reviewers, passed CI checks, and applied a three-line diff to streamline merge commit formatting. | |
| #791 docs: update CONTRIBUTING.md | craft-application | merged | Merged after reviewer approval and passing CI checks. Updated CONTRIBUTING.md and removed HACKING.md. | |
| #2568 docs: consolidate on a simple HACKING.md | snapcraft | merged | Merged into master. Consolidates development setup and contribution guidelines into a single HACKING.md file for consistency and easier onboarding. Approved by a reviewer, passed CI checks, and integrated without impacting code coverage. | |
| #5578 docs: update CONTRIBUTING.md | snapcraft | merged | Merged after approvals and passing CI. Updated CONTRIBUTING.md, removed HACKING.md, and reverted a redundant conf.py change. A reviewer noted a remaining README reference to HACKING.md requiring separate handling. | |
| #215 docs: add CONTRIBUTING.md | imagecraft | merged | Merged into main, replacing HACKING.md with CONTRIBUTING.md. Approved by reviewers and passed all CI checks. Maintainer suggested future content trimming, but the documentation update was successfully integrated. |