chore: switch to ruff
Metadata
Current evaluation
Merged PR #36 as commit ce5f6b9, switching linting and formatting to Ruff, updating configuration, and applying fixes across seven files.
Resolution: closed resolved
Reason: PR #36 was approved by two reviewers and merged as commit ce5f6b9, completing the migration to Ruff.
Evaluated by: mr-cal-bot
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #36 as commit ce5f6b9, switching linting and formatting to Ruff, updating configuration, and applying fixes across seven files. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #36 as ce5f6b9, switching linting and formatting to Ruff and updating configuration across seven files. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR switching the project's linting and formatting tool to ruff, updating configuration and applying fixes across 7 files. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #700 chore: update ruff | craft-parts | merged | Merged PR #700 updated Ruff configuration across four files, superseding #699; a follow-up lint issue was fixed in merged PR #702. | |
| #250 chore: switch to ruff snap | craft-cli | merged | Merged PR #250 as commit 35b6ab3, switching Ruff usage to its snap across three files (+5/-5); it received two approvals. | |
| #1347 chore(deps): update ruff and fix new linting errors | charmcraft | merged | Ruff pre-commit hook was bumped and resulting lint errors auto-fixed across 10 files (+40/-37). PR #1347 was merged by lengau on 2023-10-23 as commit 34018724. | |
| #115 chore(deps): update ruff and fix linting issue | craft-application | merged | Routine housekeeping PR bumping the ruff pre-commit hook and fixing the resulting lint error (2 files, +2/-4). Approved by 2 reviewers and merged into main as commit 59ab0049; Codecov's -0.10% coverage note was non-blocking. | |
| #204 chore: use ruff snap | starbase | merged | Merged PR #204 in commit 09b91ba, switching Ruff usage in four files to the Ruff snap. | |
| #192 chore: update ruff settings | starbase | merged | Merged Ruff configuration updates across two files (+6/-6) in commit d319ea9 after approval. | |
| #103 chore: fix new lints from ruff 0.12 | craft-grammar | merged | Merged PR #103 as a3ef1dd, updating Ruff to 0.12 and fixing resulting lint errors across four files; cherry-picked from canonical/starbase#409. | |
| #119 chore(deps): update dependency lint/ruff to v0.0.284 | starbase | merged | Renovate dependency bump PR #119 (lint/ruff 0.0.272 -> 0.0.284 in the ruff pre-commit hook) was approved and merged into main as commit be3ae62; +1/-1 line, no source changes. | |
| #230 chore(deps): update dependency lint/ruff to v0.1.15 | craft-cli | merged | Renovate PR merged (commit e68b0c7), bumping the ruff lint hook from 0.1.14 to 0.1.15 in .pre-commit-config.yaml; approved by two reviewers with no CI failures. | |
| #200 chore(deps): update ruff and its configuration | craft-application | merged | Merged into main via commit 07ef33dc, updating Ruff and its configuration; two reviewers approved and no unresolved comments remained. |