Skip to content

fix: silence ty 0.0.87 override checks on instrument bootstrap_config - #284

Merged
lesnik512 merged 1 commit into
mainfrom
fix/ty-0.0.87
Oct 14, 2026
Merged

lesnik512 merged 1 commit into
mainfrom
fix/ty-0.0.87

Conversation

@lesnik512

Copy link
Copy Markdown
Member

Fixes #283.

The scheduled dependency check picked up ty 0.0.87, which turns on invalid-mutable-override by default. Every framework instrument narrows bootstrap_config from its base instrument's config type, for example FastAPIConfig over LoggingConfig. That gives 25 errors.

This PR is the quick fix: one # ty: ignore[invalid-mutable-override] per line. The proper fix is to make the instruments generic over their config type. That touches every instrument, so it will get its own issue.

It also widens the build backend pin from uv_build<0.12 to uv_build>=0.11,<1.0, like the other repos. uv 0.13 warns about the old pin.

Locally with ty 0.0.87, just lint-ci is clean and just test-ci passes (362 passed, 100% coverage).

@lesnik512
lesnik512 merged commit 176cbae into main Oct 14, 2026
30 checks passed
@lesnik512
lesnik512 deleted the fix/ty-0.0.87 branch October 14, 2026 14:14
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.

Scheduled dependency check failed

1 participant