fix(lint): suppress ty
Metadata
Current evaluation
Merged after two approvals and passing CI checks. Fixes a lint failure by suppressing the ty rule across two files, modifying three lines of code.
Suggested action: —
No scores available.
Issue body
Suppresses ty rule to fix lint check
---
- [x] I've followed the [contribution guidelines](https://github.com/canonical/imagecraft/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.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after two approvals and passing CI checks. Fixes a lint failure by suppressing the ty rule across two files, modifying three lines of code. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to suppress the ty lint rule and resolve a failing lint check. The change passed make lint and make test before being approved and merged. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a change suppressing the ty lint rule to resolve failing lint checks. The update passed all lint and test validations before integration. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #242 style: fix ty linting error | craft-platforms | merged | Merged a single-file change fixing a ty linting error. Approved by one reviewer and passed CI checks. The one-line modification resolves the style violation. | |
| #5 style(lint): fix linting issues | starflow | merged | Merged after approval. Resolved linting issues by modifying one file with three additions and three deletions. No unresolved comments or CI checks were present. | |
| #446 style(lint): fix typing for ty update | craft-cli | merged | Merged a maintainer pull request fixing typing for the ty linter update. Approved by one reviewer, the change adjusted three files with minor style modifications. CI checks passed and the branch was successfully integrated. | |
| #6015 style: fix additional ty warnings | snapcraft | merged | Merged pull request resolving additional ty linter warnings for unsupported operators and non-subscriptable types. Approved by two reviewers, passed CI checks, and implemented a minimal two-file fix related to #5970. | |
| #434 style: silence ty warnings | craft-cli | merged | Merged to silence ty linter warnings, aligning with existing linter configurations. Approved by two reviewers and passed CI checks. The change modifies two files with five additions and five deletions. | |
| #1698 style: ty | craft-parts | merged | Enables ty as a lint target and fixes its warnings across 37 files (+69/-65). Fresh maintainer-authored PR, CI passing with some slow-test jobs pending, awaiting first review. | |
| #528 style(lint): ignore FBT001 and FBT002 in tests | starbase | merged | Merged. Updates lint configuration to ignore FBT001 and FBT002 rules in test files, as boolean arguments are standard testing patterns. Approved by two reviewers, passed CI checks, and applied with a minimal two-line change to align with the ecosystem. | |
| #369 chore: fix linting | imagecraft | merged | Merged after resolving redundant casting linting errors flagged by Ty. Approved by two reviewers. Changes involved minimal edits (+2/-2 lines) across one file. CI lint and test checks passed prior to merge. | |
| #429 feat(Makefile): add ty as an optional linter | starbase | merged | Merged to add ty as an optional linter in the Makefile. Approved by two reviewers and passing all CI checks, the change adds 21 lines. The linter remains non-default due to pre-production status. | |
| #1361 style: suppress ty warning | rockcraft | merged | Merged a one-line change to suppress a ty 0.0.79 false positive on user_files["shadow"]. Approved by one reviewer and passed CI checks. |