← Back to issue list

chore(lint): update ruff and config

View original Github issue

Metadata

Project
starbase
Number
#22
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged in canonical/starbase#22 via commit f3d35a6, updating Ruff and its config to ignore excessive test-function arguments (PLR0913).

Resolution: closed resolved

Reason: PR canonical/starbase#22 was merged as commit f3d35a6. It updated Ruff and allowed tests to use many fixture arguments by ignoring PLR0913.

Evaluated by: mr-cal-bot

No scores available.

Issue body

New ignore: Too many arguments on test functions

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged in canonical/starbase#22 via commit f3d35a6, updating Ruff and its config to ignore excessive test-function arguments (PLR0913).
stealth/space-bunny-alpha — closed resolved Merged canonical/starbase#22, updating Ruff and its configuration to ignore excessive-argument warnings for test functions.
qwen/qwen3.6-35b-a3b — — Merged ruff linter and configuration updates. Added an ignore rule for test functions with excessive arguments. Approved by one reviewer, modifying two files with a +4/-2 line change.
qwen3.6-35b-a3b-mtp-q6 — — Merged update to Ruff linter configuration. Added a new ignore rule to suppress warnings for test functions with too many arguments. Approved and merged without additional comments.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
84%
#588 chore: use the latest ruff config from starbase craft-application merged Merged PR #588 as commit 44b1a940, adopting starbase's latest Ruff configuration and updating linting across 86 files for the Starflow migration.
84%
#580 chore(linter): enable ruff base on starbase config craft-parts merged Merged in #580 via commit 111c8907, enabling Ruff based on the Starbase configuration.
83%
#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.
81%
#1576 chore: update ruff configs charmcraft merged Merged Ruff configuration updates across seven files in canonical/charmcraft#1576, landing as commit 764ffba3 after approval by two maintainers.
80%
#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.
80%
#192 chore: update ruff settings starbase merged Merged Ruff configuration updates across two files (+6/-6) in commit d319ea9 after approval.
79%
#568 chore(linter): rebase on starbase linters config craft-parts closed Closed unmerged after review requests; the Starbase Ruff migration was subsequently implemented more narrowly by PR #580 in commit 111c8907.
79%
#574 chore(linter): rebase on starbase linters config craft-parts closed Closed unmerged for redo; the narrower Starbase Ruff migration landed in #580 (111c8907), with autofixes in #581 (30b73d97).
78%
#37 chore(deps): update dependency ruff to v0.0.247 starbase merged Renovate PR updating ruff 0.0.245 -> 0.0.247 was approved and merged into starbase main via commit 867bb8f, bumping the ruff pin in .pre-commit-config.yaml (+1/-1).
78%