← Back to issue list

style: fix linting issues

View original Github issue

Metadata

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

Current evaluation

Merged PR #20 via commit 04cbdc0, applying style-only lint fixes across four files (+11/-5); both reviews approved and all CI checks passed.

Resolution: closed resolved

Reason: PR #20 was merged as commit 04cbdc0 after two approvals, with every lint, test, policy, and security CI check passing.

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 `make lint && make test`? ---

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #20 via commit 04cbdc0, applying style-only lint fixes across four files (+11/-5); both reviews approved and all CI checks passed.
stealth/space-bunny-alpha — closed resolved Merged PR #20 fixed linting issues across four files (+11/-5), with two reviewer approvals and all listed CI checks passing.
qwen/qwen3.6-35b-a3b — — Merged after passing all CI checks and receiving approval from two reviewers. The change resolves linting issues across four files with a +11/-5 line diff, updating code style compliance.
qwen3.6-35b-a3b-mtp-q6 — — Merged maintainer-submitted changes resolving project linting issues. Automated linting and testing checks passed before integration into the main branch.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2222 style: fix linting issues charmcraft merged Merged PR #2222 as commit 31c109b4, applying style-only lint fixes across three files (+8/-9) with two approvals.
93%
#1204 style: fix linting issues craft-parts closed Lint-only cleanup landed in commit a0622b31, matching PR #1204’s title; all reported CI checks passed.
87%
#77 lint: fix linting issues with up-to-date linters craft-store merged Merged PR #77 fixed lint failures with current linters across 11 files (+40/-24), with one approval and no unresolved review comments.
80%
#567 ci: fix linter issues craft-parts merged PR #567 merged as c41f86d6, fixing CI linter issues across four files after approval.
78%
#494 style: fix linting issues imagecraft merged Merged lint/style fixes for four files (+7/-6) in canonical/imagecraft#494 via commit a5ff7fe; the remaining lint warning was an external ubuntu.com timeout.
77%
#242 style: fix ty linting error craft-platforms merged Merged PR #242 with a one-line fix for the ty linting error; the change landed as commit e8b89ce.
77%
#20 style: fix ruff issue craft-platforms merged Merged PR #20 fixed a Ruff linting issue in one file (+1/-3); resolution is commit 71a165d.
77%
#5 style(lint): fix linting issues starflow merged Merged in canonical/starflow#5 via commit 855660d, fixing lint/style issues in one file with a +3/-3 change.
76%
#1577 style(lint): enable some previously-ignored ruff rules charmcraft merged Merged PR #1577 in commit 1b650062, enabling previously ignored Ruff rules and applying related lint/style changes across 10 files.
75%
#250 docs: fix markup rockcraft merged Merged in 4c458b4f, fixing four approved documentation markup issues across six files; remaining lint fixes were deferred to a follow-up PR.
75%