fix(lint-python): make the linter setup more resilient
Metadata
Current evaluation
Merged a fix to improve Python linter setup resilience. The change adds an apt update fallback to prevent failures when a package deb is removed. Approved by reviewers and passed all CI checks.
Suggested action: —
No scores available.
Issue body
Try setting up the linters, but if it breaks try again after an apt update.
It's written this way because mostly we don't need apt-get update and it just makes the job take longer, but when we do need it it's because a package deb got removed that makes the linter setup fail. This gets fixed automatically when GitHub rebuilds the image.
Example breakage: https://github.com/canonical/charmcraft/actions/runs/20105487204/job/57702635457?pr=2524
Example run with the fix: https://github.com/canonical/starflow/actions/runs/20109782715/job/57703659987?pr=104#step:5:38
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix to improve Python linter setup resilience. The change adds an apt update fallback to prevent failures when a package deb is removed. Approved by reviewers and passed all CI checks. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix that makes Python linter setup resilient by retrying after apt update if package installation fails, resolving intermittent CI failures. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #84 feat(lint-python): install linters in parallel | starflow | merged | Merged a single-line modification to the lint-python job that installs linters in parallel. Approved by two reviewers with all CI checks passing, the change optimizes the linting pipeline by executing setup tasks concurrently. | |
| #1154 fix(lint): ignore PLR0917 in tests alongside PLR0913 | craft-application | merged | Merged a lint fix extending the PLR0913 test ignore to also suppress PLR0917. This resolves nine lint errors in test files caused by pytest fixtures and parametrization, allowing make lint-ruff to pass. Approved by two reviewers with all CI checks passing. | |
| #77 lint: fix linting issues with up-to-date linters | craft-store | merged | Merged a maintainer-submitted pull request resolving linting errors from updated linter versions. Approved by one reviewer, passed CI, and modified 11 files (+40/-24 lines). |