Skip to content

chore(deps): upgrade ty to 0.0.84 - #4554

Closed
geoHeil wants to merge 7 commits into
2026-10-04-docs/review-guardrailsfrom
2026-10-04-chore/ty-0-0-84
Closed

geoHeil wants to merge 7 commits into
2026-10-04-docs/review-guardrailsfrom
2026-10-04-chore/ty-0-0-84

Conversation

@geoHeil

@geoHeil geoHeil commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

Upgrade the shared ty pin from 0.0.33 to 0.0.84 and regenerate uv.lock with uv 0.12.23. The lock changes only ty metadata and artifacts.

The Astral release improves type inference and performance. It also fixes GHSA-vxvm-j4xq-q7m4, which affects checks of crafted Python projects.

The newer ty exit policy now applies: warning diagnostics cause a nonzero exit. No terminal override, diagnostic suppression, or rule relaxation is added. The existing all = "warn" rule setting remains.

Local validation: the full type check reports 1075 warnings and no error-level diagnostics, then exits 1. The older 0.0.33 reports 604 warnings in the same environment and exits 0. make validate passes for the changed files. The PR fast-check smoke and 15 focused tests passed before this configuration-only removal; those checks do not cover the full warning inventory. The earlier full make check also found 11 unchanged files above its existing 1000-line limit. Awaiting CI for this head. No runtime code changes were made.

Depends on #4553, which depends on #4552.

@geoHeil

geoHeil commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor Author

@github-actions

github-actions Bot commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

✅ DCO Check Passed

Thanks @geoHeil, all your commits are properly signed off. 🎉

@geoHeil
geoHeil marked this pull request as ready for review October 4, 2026 07:22
@geoHeil
geoHeil added this pull request to stack #4555 October 4, 2026 07:26
@mergify

mergify Bot commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 Merge protection satisfied — ready to merge.

Show 1 satisfied protection

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

@geoHeil
geoHeil force-pushed the 2026-10-04-chore/ty-0-0-84 branch from 55fbd2d to ba5df77 Compare October 4, 2026 07:31
Comment thread pyproject.toml Outdated
Comment on lines +522 to +523
# Preserve the existing warning-only policy after ty changed its exit-code default.
error-on-warning = false

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets clean that up for the new standard

Signed-off-by: Georg Heiler <georg.kf.heiler@gmail.com>
Signed-off-by: Georg Heiler <georg.kf.heiler@gmail.com>
Signed-off-by: Georg Heiler <georg.kf.heiler@gmail.com>
Signed-off-by: Georg Heiler <georg.kf.heiler@gmail.com>
Signed-off-by: Georg Heiler <georg.kf.heiler@gmail.com>
Signed-off-by: Georg Heiler <georg.kf.heiler@gmail.com>
Signed-off-by: Georg Heiler <georg.kf.heiler@gmail.com>
@geoHeil
geoHeil force-pushed the 2026-10-04-chore/ty-0-0-84 branch from 5395aca to 8b80083 Compare October 4, 2026 09:30
@geoHeil

geoHeil commented Oct 4, 2026

Copy link
Copy Markdown
Contributor Author

Reparenting this PR to the reviewed cleanup stack.

@geoHeil geoHeil closed this Oct 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant