← Back to issue list

Update HACKING.md

View original Github issue

Metadata

Project
snapcraft
Number
#910
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

Merged a documentation update to HACKING.md that removes an obsolete testing section and relocates it after dependency installation. The change was approved by a reviewer with a follow-up question.

Suggested action:

No scores available.

Issue body

Removed a very obsolete section about testing, added it to the end after installing and making sure the correct set of dependencies are in place.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged a documentation update to HACKING.md that removes an obsolete testing section and relocates it after dependency installation. The change was approved by a reviewer with a follow-up question.
qwen3.6-35b-a3b-mtp-q6 Merged an update to HACKING.md that moves an obsolete testing section to the end, placing it after dependency installation instructions. Approved with a minor question.
qwen3.6-35b-a3b-mtp-q6 Merged update to HACKING.md that moves an obsolete testing section to follow dependency installation. Approved by a maintainer.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2725 Update HACKING.md snapcraft closed Closed without review or merge activity. Minor documentation update to HACKING.md.
82%
#4006 docs: update HACKING.md to recurse submodules snapcraft merged Merged documentation update to HACKING.md instructing developers to recurse into submodules for spread tests. Approved by one reviewer and merged by maintainer @mr-cal, who dismissed unrelated test failures.
79%
#564 Changed apt-get to apt in HACKING.md snapcraft merged Merged a documentation update replacing apt-get with apt in HACKING.md to align with modern Ubuntu practices. Approved by a maintainer and integrated, addressing Launchpad bug 1591421.
76%
#55 Add hacking to README.md snapcraft merged Merged a one-line README.md update adding the hacking project. Approved by sergiusens and integrated into the repository.
75%
#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.
75%
#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.
75%
#184 docs(HACKING): clarify commit header for merges starbase merged 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.
75%
#1852 Added a missing step in Hacking.md snapcraft closed Adding a missing step to HACKING.md was abandoned after a reviewer requested a bug report link and received no further updates from the contributor.
74%