← Back to issue list

lint: stricter ruff rules

View original Github issue

Metadata

Project
starbase
Number
#56
Type
pull request
State
closed
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Closed without merging. The pull request applying stricter ruff linting rules remained in pending review with no comments or CI checks and was abandoned.

Suggested action:

No scores available.

Issue body

- [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Closed without merging. The pull request applying stricter ruff linting rules remained in pending review with no comments or CI checks and was abandoned.
qwen3.6-35b-a3b-mtp-q6 Closed without merge or comments, indicating the stricter ruff linting rules were abandoned or superseded.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#264 lint: format with ruff and fix ruff linting errors rockcraft closed Closed without merging. The ruff formatting and linting fixes were abandoned due to zero reviewer engagement and a pending dependency on issue #263.
84%
#1287 style: fix ruff issues with ruff 0.13 craft-parts closed Aimed to resolve ruff 0.13 linting violations, but the pull request was closed without merging. Failing CI checks and a pending review status led to its closure.
82%
#205 style(lint): autofix ruff 0.6.4 linting errors craft-store closed Closed without merging. Abandoned due to ruff 0.6.4 ARG001 rule changes breaking the CI pipeline. No reviews were requested, checks never ran, and the PR remained inactive for 680 days.
82%
#55 lint: stricter ruff rules starbase merged Merged stricter ruff linting rules. The change modifies one file with 23 additions and 9 deletions, enforces stricter configuration, and received reviewer approval before integration.
82%
#19 style: fix linting issues for ruff 0.12 craft-artifacts closed Closed without merging. The ruff 0.12 linting fixes remained unreviewed and unmerged. CI checks for OSV-scanner, minimum dependencies, and linting continued to fail. The changes were abandoned after 393 days.
82%
#33 lint: fix lint issues with newer ruff craft-archives merged Merged to resolve linting failures introduced by a newer ruff version. Changes fix style checks across three files. Review approved, CI passed, and coverage remained unchanged. Additional unit tests were deferred to a separate tracking ticket.
79%
#139 style: stricter ruff linters starbase merged Merged stricter ruff linter rules across six files (+50/-33 lines). Approved by four reviewers, passed CI checks, and maintained 100% test coverage.
76%
#408 style: fix linting for ruff 0.12 starbase merged Merged a one-line update to resolve linting errors for ruff 0.12. Approved by two reviewers and passed all CI checks before integration.
76%
#815 fix linting issues for ruff 0.6 craft-parts merged Merged following approval from two reviewers and passing CI checks. The pull request resolves linting compatibility issues for ruff 0.6 by applying manual and automated fixes across 44 files, modifying 98 lines of code.
76%
#68 lint: update ruff and fix linting craft-archives merged Merged into main following reviewer approval and passing CI. Updates ruff configuration and resolves linting issues across four files, maintaining 100% diff coverage with no overall coverage change.
75%