← Back to issue list

docs: update CONTRIBUTING.md

View original Github issue

Metadata

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

Current evaluation

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.

Suggested action: —

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`? --- SNAPCRAFT-1166 * Update CONTRIBUTING.md * Remove HACKING.md * Revert redundant change to `exclude_patterns` in `conf.py` [Updated CONTRIBUTING.md](https://github.com/canonical/snapcraft/blob/work/update-contributing-guide/CONTRIBUTING.md)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — 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.
qwen3.6-35b-a3b-mtp-q6 — — Merged documentation update revising CONTRIBUTING.md, removing HACKING.md, and reverting a redundant conf.py change. A reviewer flagged a lingering README reference to the deleted file, but the changes were accepted and merged.
qwen3.6-35b-a3b-mtp-q6 — — Merged documentation updates replacing HACKING.md with an updated CONTRIBUTING.md and reverting a conf.py change. A reviewer noted the README still references the removed file, requiring a follow-up fix.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#791 docs: update CONTRIBUTING.md craft-application merged Merged after reviewer approval and passing CI checks. Updated CONTRIBUTING.md and removed HACKING.md.
92%
#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.
86%
#448 docs: add `CONTRIBUTING.md` template starbase merged Merged after receiving two approvals and passing all CI checks. Added a CONTRIBUTING.md template to streamline the contribution process and removed the outdated HACKING.md file.
84%
#5628 docs: update CONTRIBUTING.md snapcraft merged Merged after reviewer approval and CI validation. Updates CONTRIBUTING.md documentation. Author corrected a minor commit revert before final merge.
83%
#368 docs: update CONTRIBUTING.md craft-cli merged Merged following two reviewer approvals and successful CI checks. Updates CONTRIBUTING.md to align with the standard template, applying 107 additions and 55 deletions to a single file.
83%
#2404 docs: update `CONTRIBUTING.md` charmcraft merged Merged update to CONTRIBUTING.md incorporating changes from reviews in other repositories. Approved by three reviewers with no unresolved comments. All CI checks passed prior to merge.
82%
#826 docs: update CONTRIBUTING.md craft-application merged Merged documentation update aligning CONTRIBUTING.md with the official template. Approved by three reviewers with all CI checks passing. The change modifies one file with 104 additions and 48 deletions.
82%
#1052 docs: Update README.md contributing link rockcraft merged Merged a documentation update fixing a broken contributing link in README.md to point to CONTRIBUTING.md. Approved by three reviewers, passed CI checks, and applied as a single-line change.
81%
#841 docs: update `CONTRIBUTING.md` craft-application merged Merged after two reviewers approved and all CI checks passed. The update incorporates minor adjustments to CONTRIBUTING.md from other repository reviews, resulting in five additions and five deletions across one file.
81%
#370 docs: update `CONTRIBUTING.md` craft-cli merged Merged an update to CONTRIBUTING.md incorporating changes from other repository reviews. Approved by two reviewers with all CI checks passing. The change modified five lines across one file.
81%