← Back to issue list

style: fix linting issues for ruff 0.9

View original Github issue

Metadata

Project
craft-parts
Number
#969
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged as commit f1427855: small Ruff 0.9 lint fixes (+2/-1) across 2 files, following the ruff-pre-commit hook bump to the 0.9.x series.

Resolution: closed resolved

Reason: PR #969 was approved by two reviewers and merged into main as commit f1427855 ("style: fix linting issues for ruff 0.9 (#969)"), applying the two-line style correction so the codebase passes the newer Ruff rules introduced by the ruff-pre-commit 0.9.x updates.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged as commit f1427855: small Ruff 0.9 lint fixes (+2/-1) across 2 files, following the ruff-pre-commit hook bump to the 0.9.x series.
stealth/space-bunny-alpha — closed resolved Merged PR #969 in commit f1427855, resolving Ruff 0.9 linting issues in two files (+2/-1).
qwen/qwen3.6-35b-a3b — — Merged a minor style fix addressing linting issues for ruff 0.9. Approved by two reviewers, passed CI checks, and updated two files with a net change of one line.
qwen3.6-35b-a3b-mtp-q6 — — Merged a pull request fixing linting compatibility issues with ruff 0.9. The maintainer's changes were integrated directly into the main branch without additional review.

Update history

No update history recorded yet.

Related work

  • Related To: craft-parts#354 (confidence 70%)

    build(deps): update pre-commit hook astral-sh/ruff-pre-commit to v0.9.7 (2c41921b) — part of the Ruff 0.9 series bump whose new rules prompted these fixes

  • Related To: craft-parts#1156 (confidence 60%)

    Later equivalent PR (merged as eb82b792) fixing lint errors for Ruff 0.12.0, same recurring pattern of style-only fixes after each Ruff release

  • Related To: canonical/starbase#319 (confidence 45%)

    Sibling project PR updating lint configuration for Ruff 0.9 around the same period

Related issues

Issue Project State Summary Similarity
#1737 style: fix linting for ruff 0.5 charmcraft merged Merged PR #1737 as commit 8994523e, updating one file (+2/-6) to fix linting under Ruff 0.5.
91%
#29 style(lint): fix ruff 0.6.0 linting errors craft-platforms merged Merged PR #29 as commit 399c4e0, fixing Ruff 0.6.0 lint errors and switching the pre-commit Ruff hook to rolling versions.
90%
#1156 style: fix linting errors for ruff 0.12.0 craft-parts merged Ruff 0.12.0 lint fixes merged as PR #1156 (commit eb82b792), correcting style violations in 5 files (+9/-12) after the ruff-pre-commit v0.12.0 bump in #407; all CI checks passed.
89%
#624 style(lint): fix ruff 0.6.0 linting errors craft-providers merged Merged as commit 823622d, fixing Ruff 0.6.0 lint errors and switching the pre-commit Ruff hook to rolling versions.
89%
#173 style: fix lint for ruff 0.12 craft-archives merged Merged PR #173 as commit 53d732e, fixing lint for Ruff 0.12 while retaining Ruff 0.11 compatibility; all CI checks passed.
89%
#803 style: fix linting issues with ruff 0.12 craft-application merged Merged via commit 5e15ff09, fixing Ruff 0.12 lint failures across 14 files. A separate integration-test failure was fixed by #804.
88%
#423 style(lint): fix ruff 0.6.0 linting errors craft-application merged Merged PR #423 as commit 2f56f14b, applying ruff check --fix across 21 files (+70/-70) to clear the new Ruff 0.6.0 lint errors. Approved by 2 maintainer reviewers.
87%
#1465 style: fix linting issues for newer ruff craft-parts merged Merged #1465 as a58695ef, fixing five files (+13/-6) for compatibility with newer Ruff; all CI checks passed.
87%
#4981 style(lint): fix ruff 0.6 linting errors snapcraft merged Merged PR #4981 as commit 3a1b8a949, fixing Ruff 0.6 lint errors with a minimal one-file change.
87%
#357 style: fix lints from ruff 0.12 craft-cli merged Merged as dd635c9, fixing new Ruff 0.12 lint errors across seven files (+11/-10); all CI checks passed.
86%