← Back to issue list

docs: update CONTRIBUTING.md

View original Github issue

Metadata

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

Current evaluation

CONTRIBUTING.md was overhauled and merged in PR #5628 as commit e76eb27c6; commit 52e61041 corrected an accidental revert before merge.

Resolution: closed resolved

Reason: PR #5628 was approved and merged into main as commit e76eb27c6. The author also restored accidentally reverted guidance in follow-up commit 52e61041 before merge.

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`? --- [Updated CONTRIBUTING.md](https://github.com/canonical/snapcraft/blob/work/update-contribution-guidelines/SNAPCRAFT-1182/CONTRIBUTING.md)

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved CONTRIBUTING.md was overhauled and merged in PR #5628 as commit e76eb27c6; commit 52e61041 corrected an accidental revert before merge.
stealth/space-bunny-alpha — closed resolved Docs-only PR #5628 by jahn-junior rewrote CONTRIBUTING.md (+103/-44) for the feature-based fork workflow; merged as commit e76eb27c after follow-up commit 52e6104 fixed an accidental revert.
stealth/space-bunny-alpha — closed resolved Merged PR #5628 updated CONTRIBUTING.md (+103/-44 lines). Commit 52e6104 corrected an accidental revert before final merge.
qwen/qwen3.6-35b-a3b — — Merged after reviewer approval and CI validation. Updates CONTRIBUTING.md documentation. Author corrected a minor commit revert before final merge.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Updated CONTRIBUTING.md. The author accidentally reverted a prior commit but corrected it in a follow-up commit before the pull request was merged.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
86%
#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.
83%
#5578 docs: update CONTRIBUTING.md snapcraft merged Merged as 889bd6c79: refreshed CONTRIBUTING.md, removed HACKING.md, and reverted an unnecessary docs/conf.py change.
80%
#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.
80%
#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.
79%
#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.
78%
#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.
78%
#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.
78%
#1052 docs: Update README.md contributing link rockcraft merged Merged PR #1052 (commit 509abf1a) fixed the broken contributing link in README.md, pointing it to CONTRIBUTING.md instead of the stale URL. Docs-only, +1/-1 lines; approved by 3 reviewers.
78%
#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.
78%