docs: document tox builds
Metadata
Current evaluation
Merged following two reviewer approvals and passing CI. The update documents tox build workflows in HACKING.md and removes snapcraft_legacy from autobuild targets, resolving a previous blocking issue.
Suggested action: —
No scores available.
Issue body
- Remove snapcraft_legacy as an autobuild target, since it was blocking.
- Add temporary documentation to HACKING.md, since it likely won't remain after we impose README standards.
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox run -m lint`?
- [ ] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`)
-----
I got tired of checking my notes.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged following two reviewer approvals and passing CI. The update documents tox build workflows in HACKING.md and removes snapcraft_legacy from autobuild targets, resolving a previous blocking issue. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged documentation update adding tox build instructions to HACKING.md and removing snapcraft_legacy from autobuild targets. Changes passed code coverage validation and were integrated into the main branch. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged documentation update adding tox build instructions to HACKING.md and removing snapcraft_legacy from autobuild targets. Code coverage checks passed. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #4273 docs: fix tox doc builds | snapcraft | merged | Merged to main after reviewer approval and passing CI. The change fixes tox documentation builds by modifying a single file (+2/-12 lines), with no impact on code coverage. | |
| #502 docs: fix tox documentation builds | craft-parts | merged | Merged after PR #501 to fix tox documentation builds. The change updates one file with seven additions and two deletions, approved by two reviewers. | |
| #616 build: use tox for unit tests in CI | craft-parts | merged | Merged. CI configuration updated to use tox for unit tests. Approved by two reviewers with passing checks. Modified two files (+66/-10). | |
| #5163 tests: clean up fifos | snapcraft | merged | Merged changes clean up cached FIFOs in the .tox directory that previously broke snapcraft snap builds. Approved by three reviewers with passing CI checks. | |
| #576 build: fix tox test environments | craft-parts | merged | Merged a one-file change fixing tox test environments in the build configuration. Approved by two reviewers with zero unresolved comments. The update adjusted three lines to resolve environment setup issues. | |
| #44 docs: Tox environments and labels | starbase | merged | Merged documentation update detailing Tox environments and labels. Approved by one reviewer with zero unresolved comments. The change adds 20 lines to a single file. | |
| #4058 tox: fix linters and makefile | snapcraft | merged | Merged into main after approval and passing CI. Fixes tox linter configurations and updates the Makefile, with no impact on test coverage. | |
| #106 tox: improve UX | starbase | merged | Merged into main following reviewer approval and passing CI. Resolves #105 by improving tox UX across three files. Changes maintain 100% test coverage with no impact on overall metrics. | |
| #791 docs: update CONTRIBUTING.md | craft-application | merged | Merged after reviewer approval and passing CI checks. Updated CONTRIBUTING.md and removed HACKING.md. | |
| #477 tox: fix broken tox environments | craft-parts | merged | Merged a maintainer pull request fixing broken tox environments. The update modifies four files with eight additions and three deletions and received one reviewer approval before integration. |