ci: update PR template to use tox commands
Metadata
Current evaluation
Merged PR #4507 updated Snapcraft’s PR template to replace make and pytest commands with tox commands in commit f5964c9f7.
Resolution: closed resolved
Reason: PR #4507 was approved by two reviewers and merged; commit f5964c9f7 records the PR template update.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [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/)?
- [x] Have you successfully run `make lint`?
- [x] Have you successfully run `pytest tests/unit`?
-----
Update the PR template to use tox commands.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #4507 updated Snapcraft’s PR template to replace make and pytest commands with tox commands in commit f5964c9f7. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged snapcore/snapcraft#4507, replacing make and pytest commands in the pull request template with tox commands after approval by two reviewers. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a two-line update to the pull request template, replacing make and pytest commands with tox. Approved by two reviewers, passed CI checks, and maintained 89.22% test coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged the PR template update, replacing make lint and pytest commands with tox equivalents. Code coverage remained stable at 89.22%, and the changes were successfully integrated into the repository. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #4196 [WIP] tox CI integration improvements | snapcraft | closed | WIP PR closed unmerged with no reviews. Its tox/PR-template idea landed later as #4507 (commit f5964c9f7, 'ci: update PR template to use tox commands'). | |
| #295 ci: add tox | craft-providers | merged | Merged PR #295 in commit 9ebe593, enabling tox for craft-providers CI and resolving CRAFT-1726. | |
| #4131 tools: update tox config to latest starbase | snapcraft | merged | Merged PR #4131 as 89fd9ecb6, updating Snapcraft's tox configuration to the latest Starbase and removing obsolete tox setup. | |
| #616 build: use tox for unit tests in CI | craft-parts | merged | Merged PR #616 switched unit-test CI to tox; it was approved by two reviewers and landed as commit bd6c687f. | |
| #332 tox: improve UX for testing | craft-providers | merged | Merged PR #332 as commit 38739ea, updating tox configuration to improve the local testing experience. | |
| #4273 docs: fix tox doc builds | snapcraft | merged | Merged PR #4273 fixed tox documentation builds; the change landed on main as commit d25b49d63. |