-
Notifications
You must be signed in to change notification settings - Fork 72
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
Add python stubs for mypy #261
Comments
Thanks for your contribution but I'm working on switching to new version of generator which support annotations, so the stubs won't be needed. |
Oke thst sound good, i'm looking forward to that. I think the watch stub would still be usefull with the ParamSpec. (There is a PR as well) For now we posted these stubs to our private repo. |
#262 |
Just for reference this is what i'm using to import the stubs @borissmidt provided.
It's working ok for now. The number of types unioned with |
Yes there are indeed a lot of None types.
Om the other hand i had been bitten multiple times on a None exception
untill i handled them all :)
…On Fri, 11 Aug 2023, 19:42 Aidan, ***@***.***> wrote:
Just for reference this is what i'm using to import the stubs @borissmidt
<https://github.com/borissmidt> provided.
kubernetes-asyncio-stubs = {ref = "add-stubs", git = "https://github.com/borissmidt/kubernetes_asyncio.git", egg = "kubernetes_asyncio-stubs", subdirectory="kubernetes_asyncio-stubs"}
It's working ok for now. The number of types unioned with None make it
incredibly hard to use properly though, lots of checking if exists. But
that's more of a python typing annoyance.
—
Reply to this email directly, view it on GitHub
<#261 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABNXZFVWMBSVURQNQMUVF5DXUZVJRANCNFSM6AAAAAAZOO6H2E>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
It would be nice if this project had stubs so we get extra type safety.
Maybe these stubs can be reused?
https://github.com/MaterializeInc/kubernetes-stubs
The text was updated successfully, but these errors were encountered: