style: ty 0.0.44
Metadata
Current evaluation
Merged PR #6280 in commit 8b45efc80, applying ty 0.0.44 style updates after two approvals; unrelated CI failures did not block the merge.
Resolution: closed resolved
Reason: PR #6280 was merged as commit 8b45efc80 after approval by two reviewers. The author documented that the lint failure was unrelated and referenced #6279.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Updates for ty 0.0.44.
---
- [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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #6280 in commit 8b45efc80, applying ty 0.0.44 style updates after two approvals; unrelated CI failures did not block the merge. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #6280 (commit 8b45efc80), applying ty 0.0.44 style updates after two-reviewer approval; unrelated CI failures did not block the merge. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged update to ty version 0.0.44. Approved by two reviewers and passed core CI checks. Unrelated lint failures and integration timeouts did not block the merge. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged updates the ty dependency to version 0.0.44. All required checks passed. An unrelated lint failure was noted separately. The change was successfully integrated. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/snapcraft#6279
(confidence 95%)
The author cited this PR while explaining that the reported lint failure was unrelated.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1727 style: ty 0.0.82 | craft-parts | closed | PR #1727 updated style for ty 0.0.82 and received two approvals, but was closed unmerged after three architecture-specific Java CI jobs failed. | |
| #485 style: ty 0.0.79 | craft-cli | closed | Merged into main: a +2/-2 line one-file style fix for the new error reported by the ty 0.0.79 type checker, by maintainer mr-cal; review approved, all 25 CI checks green. Ty was later bumped to 0.0.82 in craft-parts#1727. | |
| #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. | |
| #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. | |
| #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. | |
| #910 style: fix ty error | craft-providers | merged | Merged in #910 via commit 30fc806, adding a targeted suppression for a ty false positive; all listed CI checks passed. | |
| #885 style: enable type checking with ty | craft-providers | merged | Merged as commit 28ccfe5, PR #885 enabled ty type checking and fixed resulting errors across 8 files; all CI checks passed. | |
| #938 style: ty | craft-providers | merged | Merged PR #938 as commit 44bcd1a, adding targeted ty suppressions across 8 files to ignore warnings introduced by the latest ty version. | |
| #1126 style: fix ty errors | craft-application | merged | Merged PR #1126 (commit 9f5e0e76): small +8/-10 style fix taking advantage of ty's new Pydantic support, clearing remaining type-checker errors. | |
| #1716 test: fix ty 0.0.79 issues | craft-parts | merged | Merged PR #1716 as commit 4ef1b34c, fixing ty 0.0.79 findings in unit tests across five files. |