← Back to issue list

Remove mypy and pyright

View original Github issue

Metadata

Project
craft-cli
Number
#420
Type
issue
State
closed
Author
mr-cal
Labels
Type: Cleanup Status: Triaged
Created
Updated
Closed

Current evaluation

Removed mypy and pyright in favor of ty through merged PR #479 (commit 0e49c01).

Resolution: closed resolved

Reason: PR #479, “style: drop mypy, pyright,” was merged as commit 0e49c01. The maintainer subsequently marked #420 completed.

Evaluated by: mr-cal-bot

No scores available.

Issue body

### What needs to get done Remove mypy and pyright in favor of ty. ### Why it needs to get done `ty` is replacing these type-checkers.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Removed mypy and pyright in favor of ty through merged PR #479 (commit 0e49c01).
stealth/space-bunny-alpha — closed resolved Removed mypy and pyright in favor of ty through merged PR canonical/craft-cli#479.
qwen/qwen3.6-35b-a3b — — Replaced mypy and pyright with ty for type checking. The cleanup was completed and merged via pull request #479.
qwen/qwen3.8-27b
Impact: 20
Quick Win: 15
Staleness: 55
Complexity: 25
Confidence: 88
Support Request: 0
close stale Cleanup task to remove mypy and pyright type checkers in favor of ty. PR #479 was merged removing both dependencies and switching the lint pipeline to ty; the issue is effectively complete.
qwen/qwen3.6-35b-a3b
Staleness: 55
Complexity: 15
Confidence: 85
Support Request: 0
keep open Cleanup task to remove mypy and pyright type checkers in favor of ty. Triaged by a maintainer but inactive for 141 days with only an automated Jira sync comment.
qwen3.6-35b-a3b-mtp-q6
Staleness: 70
Complexity: 25
Confidence: 85
Support Request: 0
keep open Cleanup task to remove mypy and pyright type-checkers in favor of ty. Triaged by a maintainer but inactive for 128 days with no implementation progress or further discussion.

Update history

Date Change
closed

Related work

  • Likely Fixed By: craft-cli#479 (confidence 100%)

    Merged PR that replaced mypy and pyright with ty and passed all CI checks.

Related issues

Issue Project State Summary Similarity
#6007 Remove mypy and pyright snapcraft closed Request to drop mypy and pyright for ty was resolved by merged PR #6247 ("style: go all-in on ty", commit 534d7dce7), which replaced both checkers with ty and removed the unused-ignore-comment suppression.
87%
#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.
82%
#1706 style: drop mypy, pyright craft-parts merged Merged PR #1706 in commit 770e2130, replacing mypy and pyright with ty; redundant runtime type-check cleanup was deferred to #1707.
81%
#215 style: drop mypy, pyright craft-grammar merged Merged #215 in commit 19a38b7, dropping mypy and pyright in favor of ty across six files (+4/-481).
81%
#1007 style: drop mypy, pyright craft-providers merged Merged into main as commit 232143d, replacing mypy and pyright with ty and removing obsolete type-checker tooling.
81%
#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.
79%
#901 Remove mypy and pyright craft-providers open Drop mypy/pyright in favour of ty. Makefile, CI and pyproject already use ty only; common.mk still defines lint-mypy, lint-pyright, install-pyright, and stale .gitignore/type: ignore entries remain. Mirrors snapcraft#6007 and craft-cli#420, both completed.
79%
#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.
78%
#1207 style: replace mypy and pyright with ty rockcraft merged Merged PR #1207 (commit 5d9356c1), replacing mypy and pyright type checking with Astral ty and updating lint configuration across 43 files.
76%
#1084 style: replace mypy and pyright with ty craft-application merged Merged PR #1084 as squash commit 345bd5d9, replacing mypy and pyright with ty and updating lint configuration for craft-app 7.
75%