← Back to issue list

chore(lint): update linters and fix linting issues with new rules

View original Github issue

Metadata

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

Current evaluation

Merged PR #359 in commit e7b93bde, updating linters and fixing violations introduced by the new rules across 13 files.

Resolution: closed resolved

Reason: PR #359 was approved and merged as commit e7b93bde, completing the linter update and associated lint fixes.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #359 in commit e7b93bde, updating linters and fixing violations introduced by the new rules across 13 files.
stealth/space-bunny-alpha — closed resolved Merged canonical/craft-parts#359 in commit e7b93bde, updating linters and fixing violations under the new rules across 13 files.
qwen/qwen3.6-35b-a3b — — Merged after updating linters and fixing linting issues under new rules. The change modified 13 files and was approved by one reviewer before integration.
qwen3.6-35b-a3b-mtp-q6 — — Merged an update that upgraded linting tools and resolved code violations to comply with newly enforced rules.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#585 chore(linter): misc fixes craft-parts merged Merged PR #585 with miscellaneous linter fixes across 125 files, preserving individual commits rather than squashing, as requested.
81%
#439 chore(linter): misc fix rockcraft merged Merged PR #439, applying miscellaneous linter fixes across 16 files, including a Pyright error exposed after rebasing onto main.
78%
#443 chore(linter): fix some annotation errors rockcraft merged Merged canonical/rockcraft#443 to fix linter annotation errors across 14 files (+97/-55); the maintainer-authored change was approved by two reviewers.
78%
#375 chore(lint): Remove trailing whitespace rockcraft merged Merged PR #375 removed trailing whitespace from one file (+4/-4), resolving yamllint complaints; landed as commit 777df7e3.
78%
#1099 chore: fix linting craft-application merged Merged as 4c674c21, updating two files to clear Ty unused-directive lint errors; lint/files passed and review was approved.
77%
#369 chore: fix linting imagecraft merged Merged PR #369 removed two redundant casts (1 file, +2/-2), fixing Ty linting; the lint/files check passed.
76%
#345 chore: hush some of the linter output starbase merged Merged PR #345 quieted noisy linter tasks so earlier lint steps no longer flooded terminal output; landed as commit c9e090a.
76%
#6431 chore(merge): 9.1.2 into main snapcraft closed Merged release branch 9.1.2 into main via commit 264420ba4; the remaining lint/files failure was attributed to an unrelated ty update.
76%
#364 chore(deps): update dependencies rockcraft merged Merged dependency updates in PR #364 via commit 1661d115, changing seven files (+152/-174) after approval by two reviewers.
76%
#183 chore: make pre-commit linters use lint group for renovate starbase merged Merged PR #183 in commit f6db1eb, updating pre-commit configuration so Renovate-managed linters use the lint dependency group.
76%