← Back to issue list

style: disable pyright in favour of ty

View original Github issue

Metadata

Project
craft-platforms
Number
#207
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged as f287a56, replacing the active Pyright configuration with Astral’s ty type checker.

Resolution: closed resolved

Reason: PR #207 was merged as commit f287a56 with all reported CI checks passing. The change disabled Pyright in favour of ty.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `make lint && make test`? ---

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged as f287a56, replacing the active Pyright configuration with Astral’s ty type checker.
qwen/qwen3.6-35b-a3b — closed resolved Merged PR disabling pyright in favor of ty for Python type checking, updating lint configuration and CI checks.

Update history

Date Change
created

Related work

  • Related To: craft-platforms#223 (confidence 90%)

    Later commit 23f2017 switched the remaining mypy+pyright setup to ty.

Related issues

Issue Project State Summary Similarity
#152 style: remove mypy and pyright to go all-in on ty debcraft merged Merged canonical/debcraft#152, removing mypy and pyright and standardizing type checking on Astral ty after approval by two reviewers.
81%
#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.
80%
#421 style: drop mypy, pyright craft-store merged Merged #421 to replace mypy and pyright with Astral ty; the lint pipeline now runs lint-ty.
80%
#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.
74%
#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.
73%
#472 style: enable ty by default starbase merged Merged PR #472 as commit 7982765, making Astral ty a default type-checking option while retaining existing checkers.
72%
#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.
72%
#186 style(typing): update pyright craft-application merged Merged PR #186 as commit 43e6f1b9, updating Pyright configuration and typing across seven files (+29/-20).
72%
#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).
72%
#158 style(types): update pyright craft-store merged Merged into main as commit 1b66727, updating Pyright typing configuration in two files after approval by two reviewers.
72%