fix(tox): use requirements-devel for linters
Metadata
Current evaluation
Merged a pull request updating the tox configuration to use requirements-devel for linters. Approved by two reviewers and passing CI, the change modifies one file to resolve linting dependencies.
Suggested action: —
No scores available.
Issue body
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] 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`)
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a pull request updating the tox configuration to use requirements-devel for linters. Approved by two reviewers and passing CI, the change modifies one file to resolve linting dependencies. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix to update the tox configuration, switching linter execution to requirements-devel. The change was accepted and integrated into the main branch. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix updating tox to use requirements-devel for linters. The change standardizes linting dependencies and was integrated into the main branch. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. | |
| #67 lint: fix ruff linting in tox | craft-archives | merged | Merged after reviewer approval and passing CI. Resolves ruff linting issues in the tox environment by updating three files. Code coverage remains unchanged at 88.07%. | |
| #66 tools: fix ruff linting command in tox | starbase | merged | Merged a one-line fix that corrected the ruff linting command in tox. The change received reviewer approval and was successfully integrated into the codebase. | |
| #47 Fix tox environment setup | craft-cli | merged | Merged a fix for the tox environment setup by removing outdated references to deleted requirements files from tox.ini. Approved by two reviewers and successfully integrated. | |
| #1755 fix(ci): fix tox ensure_version_matches | charmcraft | merged | Merged after approval by two reviewers. Resolved a CI failure in the tox ensure_version_matches check with a single-line adjustment. The fix was successfully integrated into the codebase. | |
| #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. | |
| #102 tox: pass CI environment variable | starbase | merged | Merged after approval by two reviewers and passing CI checks. The change upstreams a modification to pass the CI environment variable to tox, adding two lines to one file. | |
| #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). | |
| #1674 fix: make tox tests succeed | charmcraft | merged | Merged to fix failing tox tests by correcting an environment variable and working around canonical/operator#1212. Approved by two reviewers with a 7-line addition and 2 deletions across three files. |