← Back to issue list

lint: fix ruff linting in tox

View original Github issue

Metadata

Project
craft-archives
Number
#67
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

PR #67 was merged as commit 0148023, fixing Ruff linting configuration in tox.

Resolution: closed resolved

Reason: Merged PR canonical/craft-archives#67 resolves the lint setup as commit 0148023; the approved change had no unresolved review comments.

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 PR #67 was merged as commit 0148023, fixing Ruff linting configuration in tox.
stealth/space-bunny-alpha — closed resolved Merged as commit 0148023: fixed Ruff linting in tox and updated lint configuration across three files.
stealth/space-bunny-alpha — closed resolved Merged PR #67 as commit 0148023, fixing Ruff linting in tox across three files; coverage remained at 88.07%.
stealth/space-bunny-alpha — closed resolved Merged canonical/craft-archives#67 (c322b37), fixing Ruff linting under tox by updating three files; coverage remained unchanged.
qwen/qwen3.6-35b-a3b — — Merged after reviewer approval and passing CI. Resolves ruff linting issues in the tox environment by updating three files. Code coverage remains unchanged at 88.07%.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#66 tools: fix ruff linting command in tox starbase merged Merged #66 as commit 189835b, correcting the one-line Ruff lint invocation in tox.
92%
#4416 style: fix ruff format tox environment snapcraft merged Merged #4416 via commit 927f66c04, correcting the Ruff formatter's tox environment configuration.
85%
#4058 tox: fix linters and makefile snapcraft merged PR #4058 by lengau merged as commit be5d4307 ("tox: fix linters and makefile"), correcting linter config in tox.ini and the Makefile after the starbase tox migration (#4046). Codecov reported no coverage change.
76%
#68 lint: update ruff and fix linting craft-archives merged Merged PR #68 as commit 8bb81ae, updating Ruff and fixing lint violations across four files after prerequisite #67.
75%
#4820 fix(tox): use requirements-devel for linters snapcraft merged PR #4820 merged as commit e6a0f5c3a, adding 4 lines to tox.ini so the lint environment installs linter dependencies from requirements-devel instead of an ad-hoc list. tox.ini was later deleted (18a7596c9).
75%
#4981 style(lint): fix ruff 0.6 linting errors snapcraft merged Merged PR #4981 as commit 3a1b8a949, fixing Ruff 0.6 lint errors with a minimal one-file change.
74%
#1737 style: fix linting for ruff 0.5 charmcraft merged Merged PR #1737 as commit 8994523e, updating one file (+2/-6) to fix linting under Ruff 0.5.
74%
#4273 docs: fix tox doc builds snapcraft merged Merged PR #4273 fixed tox documentation builds; the change landed on main as commit d25b49d63.
73%
#357 style: fix lints from ruff 0.12 craft-cli merged Merged as dd635c9, fixing new Ruff 0.12 lint errors across seven files (+11/-10); all CI checks passed.
73%
#803 style: fix linting issues with ruff 0.12 craft-application merged Merged via commit 5e15ff09, fixing Ruff 0.12 lint failures across 14 files. A separate integration-test failure was fixed by #804.
73%