← Back to issue list

starbase ruff

View original Github issue

Metadata

Project
snapcraft
Number
#4816
Type
pull request
State
closed
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Abandoned without merge. The pull request updated ruff configuration, removed isort, and applied autofixes across 125 files.

Suggested action:

No scores available.

Issue body

- **style: update ruff configuration** - **style: clear some ignored ruff linters** - **style: remove isort** - **style: update ruff config to latest starbase** - **style: autofix with ruff** - **fix: fix issues introduced by autoformat**

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Abandoned without merge. The pull request updated ruff configuration, removed isort, and applied autofixes across 125 files.
qwen3.6-35b-a3b-mtp-q6 Closed after merging ruff configuration updates, removing isort, and autofixing code to align with the latest starbase standards. Linting issues were resolved.
qwen3.6-35b-a3b-mtp-q6 Closed after merging Ruff configuration updates, removing isort, and applying autofixes. Style changes were integrated and autoformat issues resolved.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1232 tools: update ruff configuration to starbase v1 charmcraft merged Merged after two reviewer approvals. Updated ruff configuration to starbase v1 across 64 files (+598/-525 lines). Changes resolved a circular dependency and streamlined tooling.
80%
#415 style: fix ruff 0.16 warning craft-store merged Merged after two approvals and passing CI. Resolved a ruff 0.16 warning by adding an ignore directive copied from Starbase. The change added a single line to one file.
74%
#581 chore(linter): ruff auto fix base on starbase config craft-parts merged Merged automated ruff fixes applying the starbase config across 156 files. Adjusts formatting and import order, with tweaks for trailing commas and ignored rules. Approved by three reviewers and depends on PR #580.
73%
#2074 style: update to ruff 0.9.0 formatting charmcraft merged Merged to align code formatting with ruff 0.9.0 linting rules. Approved by two reviewers and passed CI. One change was noted for upstreaming to starbase instead of applying it locally.
72%
#4712 Move more linting to ruff snapcraft closed Completed. Linting configuration was updated to align with starbase, replacing isort with ruff and disabling redundant flake8 and pylint tests. Maintainer confirmed the changes and closed the issue.
71%
#95 Merge starbase craft-archives closed Merged into main. Integrated the starbase branch with ruff formatting, disabled mandatory fixture() checks, and applied manual fixes across 25 files. Coverage improved by 0.51% with 80% diff coverage.
71%
#5574 style: ruff 0.12 snapcraft merged Merged after two reviewer approvals. Updates codebase for ruff 0.12 compatibility. CI passed except a non-blocking OSV-scanner failure. Changes affect 8 files with 56 additions and 23 deletions.
70%
#580 style(ruff): fix linting issues from ruff update craft-application merged Merged after approval by three reviewers. Resolves linting errors from a ruff update, modifying three files with a +7/-7 line change. Successfully integrated into the codebase.
70%