← Back to issue list

chore(deps): update ruff to 0.1.1 and fix lint issues

View original Github issue

Metadata

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

Current evaluation

Ruff 0.1.1 dependency bump plus lint fixes merged to snapcraft main as commit 4b1cff546 (PR #4417, +66/-36 across 19 files). A later follow-up, style: ruff 0.16 (#582), superseded the pin.

Resolution: closed resolved

Reason: The PR was approved by two reviewers and squash-merged into snapcraft's main branch as commit 4b1cff546 ("chore(deps): update ruff to 0.1.1 and fix lint issues (#4417)"), updating the Ruff pin in pyproject.toml and fixing the resulting lint violations. Maintainer lengau noted the subsequent 0.1.2 release was a trivial follow-up.

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 Ruff 0.1.1 dependency bump plus lint fixes merged to snapcraft main as commit 4b1cff546 (PR #4417, +66/-36 across 19 files). A later follow-up, style: ruff 0.16 (#582), superseded the pin.
stealth/space-bunny-alpha — closed resolved Merged in snapcore/snapcraft#4417, upgrading Ruff to 0.1.1 and fixing lint violations across 19 files in commit 4b1cff546.
qwen/qwen3.6-35b-a3b — — Merged dependency update upgrading ruff to 0.1.1 and resolving linting errors across 19 files. Approved by two reviewers, passed CI, and increased coverage by 0.01%. A newer ruff version was released shortly after merge.
qwen3.6-35b-a3b-mtp-q6 — — Merged dependency update to ruff 0.1.1 with lint fixes. A maintainer noted ruff 0.1.2 was released and confirmed the next update will be straightforward. Merge increased coverage by 0.01%.

Update history

No update history recorded yet.

Related work

  • Superseded By: snapcore/snapcraft#582 (confidence 70%)

    Later commit adddfde80 'style: ruff 0.16' bumps the same linter far beyond 0.1.1, superseding this change.

  • Related To: snapcore/snapcraft#6443 (confidence 25%)

    Later dependency-update PR touching the same toolchain, indicating the 0.1.1 pin was short-lived.

Related issues

Issue Project State Summary Similarity
#595 chore(deps): update dependency lint/ruff to v0.5.1 craft-providers closed Renovate PR to bump lint/ruff 0.4.10 → 0.5.1 was closed unmerged; maintainer @lengau closed it as "superseded by #597", which replaced the pinned ruff dep with the ruff snap.
92%
#205 chore(deps): update dependency lint/ruff to v0.2.1 - autoclosed craft-application closed Renovate PR #205 was auto-closed unmerged; PR #206 replaced its Ruff package pin with Ruff from the Snap.
91%
#217 chore(deps): update dependency lint/ruff to v0.1.8 craft-cli merged Renovate PR bumping lint/ruff 0.1.7 to 0.1.8 was approved by two reviewers and merged into main as commit 61f7791 (Dec 2023); later superseded by ruff 0.1.14 (#223) and 0.1.15 (#230).
91%
#566 chore(deps): update dependency lint/ruff to v0.4.3 craft-providers merged Renovate PR #566 bumped the lint/ruff pin from ==0.4.2 to ==0.4.3 (+1/-1 line). Approved by @mr-cal (who noted ruff would move to a snap at the next Starbase merge) and merged as commit e7b2ddc.
90%
#142 chore(deps): update dependency lint/ruff to v0.1.1 starbase merged Merged PR #142 via commit 6b79ebe, updating the pinned Ruff dependency from v0.1.0 to v0.1.1.
90%
#114 chore(deps): update dependency lint/ruff to v0.1.0 craft-application closed Renovate PR bumping lint/ruff 0.0.269 -> 0.1.0 was closed unmerged by maintainer @lengau as "Superceded by #115", which updated Ruff and fixed a linting issue.
90%
#589 chore(deps): update dependency ruff to v0.1.14 craft-parts merged Renovate PR bumping ruff0.1.6 to 0.1.14 was approved and auto-merged into main (commit 210fd091), a one-line dependency change in the lint requirements.
90%
#115 chore(deps): update dependency lint/ruff to v0.1.1 craft-store merged Renovate PR #115 was merged, bumping lint/ruff from 0.1.0 to 0.1.1 in commit 1f36381.
90%
#204 chore(deps): update dependency lint/ruff to v0.1.6 craft-cli merged Renovate PR #204 merged, upgrading lint/ruff from 0.1.0 to 0.1.6 after two approvals; the dependency pin changed by one line.
90%
#118 chore(deps): update dependency lint/ruff to v0.1.1 craft-application closed Renovate PR bumping lint/ruff 0.0.269→0.1.1 was closed unmerged; maintainer lengau marked it "Superseded by #115", which updated Ruff and fixed the resulting lint errors and was merged into main.
90%