-
-
Notifications
You must be signed in to change notification settings - Fork 118
Third-party tests failed on Thu Apr 17 2025 #586
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Old dirty code from Pydantic v1, I'll fix it. |
Thanks! This would still mean that future releases of typing-extensions will be incompatible with any currently released version of Pydantic, right? It might be worth re-adding |
Only Pydantic V1, so the impact is limited (and while we still rely on a couple private |
Ah right, it would only be if users explicitly import Looks like litestar also relies on I feel like that's enough reason to re-add the private symbol to typing-extensions (though I'd still encourage you to avoid using it). |
Mh. We use it to check if something is
but that is brittle. Many things can have a I'd also prefer not to use undocumented private types, but currently don't know of a better alternative. |
Like this:
|
@provinzkraut you can also rely on our |
@provinzkraut we also have some docs on robust strategies to introspect typing constructs here: https://typing-extensions.readthedocs.io/en/latest/#runtime-use-of-types |
Heh, thanks for the reminder. This comment made me re-visit our code again, and it turns out the util that has failed in the run above isn't actually used anymore, and we've migrated away from using We should be able to just remove the offending code, so there will be no compatibility issue from our side anymore. |
Great, thanks! I think it still makes sense to keep the name defined in typing-extensions for some time so users can safely upgrade typing-extensions without having to worry about upgrading other libraries, and because there are surely more people using |
Run listed here: https://github.com/python/typing_extensions/actions/runs/14506991058
The text was updated successfully, but these errors were encountered: