Run pylint in Github CI.
Metadata
Current evaluation
Merged PR #59 added pylint to GitHub CI via ad06d5c (merge 6dafc86); later PR #173 replaced pylint with Ruff.
Resolution: closed resolved
Reason: PR #59 was merged into main as 6dafc86, with commit ad06d5c adding pylint to GitHub Actions. Its lint setup was later superseded by PR #173.
Evaluated by: mr-cal-bot
No scores available.
Issue body
No description provided.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #59 added pylint to GitHub CI via ad06d5c (merge 6dafc86); later PR #173 replaced pylint with Ruff. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #59 added Pylint to GitHub CI through a three-line workflow change (commit ad06d5c); merge commit: 6dafc86. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #59 added pylint to GitHub CI through a three-line configuration change, receiving two approvals (commit ad06d5c). | |
| qwen/qwen3.6-35b-a3b | — | — | Merged to add pylint to GitHub CI. Approved by two reviewers with no comments. The change adds three lines to one configuration file, enabling automated static code analysis during continuous integration. |
Update history
No update history recorded yet.
Related work
-
Superseded By:
canonical/craft-cli#173
(confidence 98%)
Merged later to enable Ruff, remove pylint, and update related lint checks.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #565 ci: pin pylint version | craft-parts | merged | Merged PR #565, commit 174d0e88, pinned pylint in CI for stability; migration to Ruff was deferred. | |
| #675 style(lint): replace pylint with ruff | rockcraft | merged | Merged as b7d70598, replacing pylint with Ruff, adding Ruff to CI and project requirements, and enabling equivalent lint coverage. | |
| #562 Remove pylint in favour of ruff | craft-parts | closed | Pylint was dropped in favour of ruff by merged PR #580 (commit 111c8907, "chore(linter): enable ruff base on starbase config"), removing the [tool.pylint] config and CI/Makefile/dependency usage; ruff config expanded to cover it. | |
| #118 style: remove pylint | craft-store | merged | Merged PR #118 as commit beb9201, removing Pylint configuration and tooling after Ruff had replaced it. | |
| #42 pylint shows some warnings | craft-cli | closed | Pylint warnings were fixed and pylint was enforced in GitHub CI by merged PR #59 (commit ad06d5c; merge 6dafc86). |