style: suppress ty diagnostics that block CI
Metadata
Current evaluation
Merged PR #2961 adds ty-specific lint ignores to match existing pyright/mypy configurations and removes a redundant cast in _expand_base, resolving CI blocks.
Resolution: closed resolved
Reason: The pull request was merged, introducing ty-specific diagnostic ignores aligned with existing pyright/mypy settings and cleaning up a redundant cast, which resolved the CI blocking issue.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Recreate of #2957 by lengau. Adds ty-specific ignores matching existing pyright/mypy ignores; removes redundant cast in _expand_base.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR #2961 adds ty-specific lint ignores to match existing pyright/mypy configurations and removes a redundant cast in _expand_base, resolving CI blocks. |
Update history
| Date | Change |
|---|---|
| merged | |
| updated | |
| updated | |
| updated | |
| updated | |
| created |
Related work
-
Related To:
charmcraft#2957
(confidence 95%)
This PR is explicitly noted as a recreation of #2957 by lengau.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2957 style: suppress ty diagnostics that block CI | charmcraft | closed | Merged PR #2957 to fix CI `lint-ty` failures by suppressing 22 diagnostics and fixing a redundant cast. | |
| #6247 style: go all-in on ty | snapcraft | merged | Merged PR #6247 (commit 534d7dce7) replaced mypy and pyright with ty, removed obsolete suppressions, and resolved #6007. | |
| #335 fix(lint): suppress ty | imagecraft | merged | Merged in PR #335 as c270f0a, adding targeted ty suppressions in two files so lint passes; all CI checks passed. | |
| #253 style: drop mypy, pyright | craft-archives | merged | Merged PR #253 (commit 1949f53) drops mypy and pyright in favor of ty; lint and test CI checks passed. | |
| #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. | |
| #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. | |
| #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. | |
| #27 ci: type check with ty | craft-artifacts | merged | Merged PR #27 as commit 716f05a, adding ty type checking to CI and suppressing intentional type violations; related lint fixes landed in #20. | |
| #479 style: drop mypy, pyright | craft-cli | merged | Merged PR #479 (commit 0e49c01) dropped the mypy and pyright type checkers in favour of ty, removing ~340 lines of type-checker config and ignore directives across 8 files; all CI checks passed. | |
| #434 style: silence ty warnings | craft-cli | merged | Merged PR #434 as commit ef91a65, adding targeted ty suppressions matching existing suppressions for other linters across two files. |