← Back to issue list

build: update ruff settings

View original Github issue

Metadata

Project
rockcraft
Number
#481
Type
pull request
State
closed
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Ruff lint-config tweak PR closed unmerged; maintainer lengau closed it 2024-08-28 as superseded by #675 ("style(lint): replace pylint with ruff", merged as commit b7d70598).

Resolution: closed superseded

Reason: Closing comment from lengau on 2024-08-28 states the PR "is superseded by #675"; that PR replaced pylint with Ruff and was merged as b7d70598, covering the same lint configuration changes (diff was +6/-6 in 1 file).

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed superseded Ruff lint-config tweak PR closed unmerged; maintainer lengau closed it 2024-08-28 as superseded by #675 ("style(lint): replace pylint with ruff", merged as commit b7d70598).
stealth/space-bunny-alpha — closed superseded Closed unmerged as superseded by canonical/rockcraft#675, which replaced pylint with Ruff and was merged as b7d70598.
qwen/qwen3.6-35b-a3b — — Superseded by pull request #675. The ruff configuration update was replaced by a newer change and closed without merging.
qwen3.6-35b-a3b-mtp-q6 — — Superseded by pull request #675. The ruff settings updates were replaced by the newer PR, which was merged to implement the changes.

Update history

No update history recorded yet.

Related work

  • Superseded By: canonical/rockcraft#675 (confidence 98%)

    Named explicitly in the closing maintainer comment; merged as b7d70598 replacing pylint with Ruff.

  • Related To: canonical/rockcraft#363 (confidence 45%)

    Related earlier Ruff configuration work (removed black in favour of ruff), part of the same lint-tooling migration line.

Related issues

Issue Project State Summary Similarity
#660 build: update ruff settings craft-parts closed Ruff configuration PR was closed unmerged after prolonged inactivity; its six-line settings update was not applied.
84%
#201 build: update ruff settings craft-application closed Ruff-settings update was approved but PR #201 was closed unmerged after about 2.6 years without CI activity or follow-up.
82%
#319 style: update linting configuration for ruff 0.9 starbase merged Ruff 0.9 lint configuration update merged into main via commit 45ab05b (PR #319). The unrelated git_describe_command suggestion was deferred to PR #321.
81%
#510 build: update ruff settings craft-providers merged Merged PR #510 updated the Ruff configuration and was committed as bc34c1c.
80%
#1517 build: update ruff settings charmcraft merged Merged into charmcraft as commit 33854f04, updating the project's Ruff lint configuration (+6/-6).
80%
#19 style: fix linting issues for ruff 0.12 craft-artifacts closed PR #19 was closed unmerged after prolonged inactivity; its Ruff 0.12 lint fixes were abandoned. Later PR #20 was merged as commit 04cbdc0.
79%
#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.
79%
#907 style: adjust linting rules for ruff 0.12 rockcraft merged Merged into main as 186cb0fb, updating Ruff 0.12 settings to ignore PLC0415 and avoid unnecessary noqa comments.
78%
#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.
78%
#33 lint: fix lint issues with newer ruff craft-archives merged PR #33 merged as commit c2a6687, fixing new lint failures introduced by a newer Ruff release across3 files (+5/-3). Maintainer deferred extra unit tests to CRAFT-1641.
78%