← Back to issue list

docs: update `CONTRIBUTING.md`

View original Github issue

Metadata

Project
snapcraft
Number
#5639
Type
pull request
State
merged
Author
jahn-junior
Labels
—
Created
Updated
Closed

Current evaluation

Merged docs-only PR updating CONTRIBUTING.md (+16/-18) to match Canonical cross-repo template changes, landing as commit 87afdc5b7 with 2 approvals; only store spread job failed.

Resolution: closed resolved

Reason: PR #5639 was merged into main as commit 87afdc5b7 ("docs: update CONTRIBUTING.md (#5639)"), confirmed in the CONTRIBUTING.md commit history. The maintainer held it in draft until the template stabilised across repos, then merged after two approvals with zero unresolved review comments.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [X] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)? - [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [X] Have you successfully run `make lint`? - [X] Have you successfully run `make test`? --- Updates `CONTRIBUTING.md` to reflect template changes that resulted from its adoption in other repos.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged docs-only PR updating CONTRIBUTING.md (+16/-18) to match Canonical cross-repo template changes, landing as commit 87afdc5b7 with 2 approvals; only store spread job failed.
stealth/space-bunny-alpha — closed resolved Updated CONTRIBUTING.md for cross-repository template changes in merged PR #5639 after two approvals; all checks except integration-spread-tests-store passed.
stealth/space-bunny-alpha — closed resolved Merged PR #5639, commit 87afdc5b7, updated CONTRIBUTING.md to align with template changes adopted in other repositories.
qwen/qwen3.6-35b-a3b — — Merged after updating CONTRIBUTING.md to align with cross-repository template changes. Kept in draft until template stabilized, then approved by two reviewers and merged despite one failing CI check.
qwen3.6-35b-a3b-mtp-q6 — — Merged CONTRIBUTING.md updates aligning with cross-repository template changes. Held in draft until stabilization, then merged following cross-team review and a final sanity check.

Update history

No update history recorded yet.

Related work

  • Related To: snapcraft#5637 (confidence 80%)

    Follow-up typo/link fixes to CONTRIBUTING.md in commit 4ad38c556, part of the same template rollout

  • Related To: snapcraft#5628 (confidence 75%)

    Earlier larger CONTRIBUTING.md template update (+103/-44) in the same series preceding #5639

  • Related To: snapcraft#5578 (confidence 70%)

    Earlier CONTRIBUTING.md refresh (commit 889bd6c79) that introduced the new template and removed HACKING.md

  • Related To: snapcraft#6111 (confidence 55%)

    Later downstream docs commit 157e26f19 removing bounty instructions from the same template

Related issues

Issue Project State Summary Similarity
#2404 docs: update `CONTRIBUTING.md` charmcraft merged Merged docs PR #2404 as commit 1c0f9884, applying a small +5/-5 edit to CONTRIBUTING.md carrying over review feedback from other Canonical repos; 3 approvals, all CI green.
87%
#826 docs: update CONTRIBUTING.md craft-application merged PR #826 merged as commit 23a2d682, aligning CONTRIBUTING.md with Canonical's contribution template (+104/-48). Merged after 3 approvals with all CI checks green; applied after prerequisite PR #825.
85%
#841 docs: update `CONTRIBUTING.md` craft-application merged Merged PR #841, applying five CONTRIBUTING.md line replacements from reviews in other Canonical repositories; all CI checks passed.
83%
#299 docs: update `CONTRIBUTING.md` craft-store merged Merged a documentation-only CONTRIBUTING.md update (+107/-52) as commit cabdfe3 after three approvals; tests, lint, and documentation checks passed despite two failing OSV-scanner jobs.
82%
#368 docs: update CONTRIBUTING.md craft-cli merged Merged PR #368 (commit e75f8ef) updated CONTRIBUTING.md to align with the project template, +107/-55 lines in one file; all CI checks passed.
80%
#370 docs: update `CONTRIBUTING.md` craft-cli merged Merged PR #370 (commit b7a0dbb), aligning CONTRIBUTING.md with the project template and incorporating review wording changes.
79%
#2382 docs: update `CONTRIBUTING.md` charmcraft merged Docs-only PR by maintainer jahn-junior merged as commit 08bdaed6 (+103/-60), rewriting CONTRIBUTING.md; follow-up fixes landed in #2392 (make setup system packages) and #2404.
78%
#1142 docs: maintain contributing.md craft-parts merged Merged docs PR by maintainer medubelko fixing an error and improving a step in CONTRIBUTING.md (+3/-4 lines). Approved by 2 reviewers, all CI green; landed as commit 14b47207 ('docs: maintain contributing.md (#1142)').
77%
#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.
76%
#5628 docs: update CONTRIBUTING.md snapcraft merged CONTRIBUTING.md was overhauled and merged in PR #5628 as commit e76eb27c6; commit 52e61041 corrected an accidental revert before merge.
76%