← Back to issue list

style: ty 0.0.87, ruff 0.17.0

View original Github issue

Metadata

Project
snapcraft
Number
#6493
Type
pull request
State
merged
Author
mr-cal
Labels
PR: Squash
Created
Updated
Closed

Current evaluation

Merged PR updating ty to 0.0.87 and ruff to 0.17.0, with 7 AI-assisted commits fixing linting errors across 12 files.

Resolution: closed resolved

Reason: PR #6493 was merged, updating linting tools and fixing associated errors. Approved by 2 reviewers with CI passing on most checks.

Evaluated by: mr-cal-bot

No scores available.

Issue body

7 AI-assisted commits, each fixing a different set of errors from ty 0.0.87 and ruff 0.17.0. --- - [x] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md). - [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [x] I've successfully run `make lint && make test`. - [ ] I've added or updated any relevant documentation. - [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — closed resolved Merged PR updating ty to 0.0.87 and ruff to 0.17.0, with 7 AI-assisted commits fixing linting errors across 12 files.

Update history

Date Change
merged
updated
updated
updated
created

Related issues

Issue Project State Summary Similarity
#6498 style: ty 0.0.87, ruff 0.17.0 snapcraft merged Merged PR #6498, cherry-picking style fixes from #6493 onto hotfix/9.2 to update ty to 0.0.87 and ruff to 0.17.0.
83%
#1156 style: fix linting errors for ruff 0.12.0 craft-parts merged Ruff 0.12.0 lint fixes merged as PR #1156 (commit eb82b792), correcting style violations in 5 files (+9/-12) after the ruff-pre-commit v0.12.0 bump in #407; all CI checks passed.
76%
#1698 style: ty craft-parts merged Merged PR #1698 as commit 6b4db96f, enabling ty as a lint target and fixing its warnings across 37 files.
76%
#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.
75%
#2074 style: update to ruff 0.9.0 formatting charmcraft merged Merged PR #2074 in commit af377cd8, updating formatting for Ruff 0.9.0 to address new lint failures.
74%
#173 style: fix lint for ruff 0.12 craft-archives merged Merged PR #173 as commit 53d732e, fixing lint for Ruff 0.12 while retaining Ruff 0.11 compatibility; all CI checks passed.
74%
#6280 style: ty 0.0.44 snapcraft merged Merged PR #6280 in commit 8b45efc80, applying ty 0.0.44 style updates after two approvals; unrelated CI failures did not block the merge.
74%
#815 fix linting issues for ruff 0.6 craft-parts merged Merged PR #815, applying Ruff 0.6 autofixes and manual lint corrections across 44 files (+98/-98).
74%
#1396 style: fix findings from ruff 0.17, shfmt, and tombi rockcraft merged Merged PR fixing linting and style findings from ruff 0.17, shfmt, and tombi, plus adding ty diagnostic suppressions for intentional pydantic field narrowing.
74%
#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.
74%