← Back to issue list

lint: reintroduce pylint in tox

View original Github issue

Metadata

Project
snapcraft
Number
#4073
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged into main as edd11ada8, reintroducing pylint to the tox lint environment after the tox migration.

Resolution: closed resolved

Reason: PR #4073 was approved and merged into main. Resolving commit edd11ada8 restored pylint in tox.

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`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged into main as edd11ada8, reintroducing pylint to the tox lint environment after the tox migration.
stealth/space-bunny-alpha — closed resolved Merged into main, restoring pylint in the tox lint environment; the change landed as commit edd11ada8.
stealth/space-bunny-alpha — closed resolved Merged PR #4073 reintroduced Pylint to Snapcraft's tox lint environment in commit edd11ada8.
stealth/space-bunny-alpha — closed resolved PR #4073 was merged into main, restoring pylint in tox; the change landed as commit edd11ada8.
stealth/space-bunny-alpha — closed resolved Merged PR #4073 as commit edd11ada8, reintroducing pylint to the tox lint configuration; code coverage remained unchanged.
stealth/space-bunny-alpha — closed resolved Merged into main as commit edd11ada8, reintroducing pylint in the tox configuration.
stealth/space-bunny-alpha — closed resolved Merged into main (#4073, 36d549b), reintroducing pylint in the tox configuration with a 15-line, 1-line deletion change to one file; coverage was unchanged.
qwen/qwen3.6-35b-a3b — — Merged into main following approval and passing CI checks. The change reintroduces pylint to the tox configuration, adding 15 lines and removing 1 in a single file. Code coverage remains unchanged.
qwen3.6-35b-a3b-mtp-q6 — — Merged into main, reintroducing pylint in tox for project linting. Codecov reported no coverage impact, successfully restoring static analysis tooling to the build configuration.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#67 lint: fix ruff linting in tox craft-archives merged PR #67 was merged as commit 0148023, fixing Ruff linting configuration in tox.
72%
#4820 fix(tox): use requirements-devel for linters snapcraft merged PR #4820 merged as commit e6a0f5c3a, adding 4 lines to tox.ini so the lint environment installs linter dependencies from requirements-devel instead of an ad-hoc list. tox.ini was later deleted (18a7596c9).
71%