-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Work on merging annotations from types-pika (a non-typeshed project) into typeshed's pika stubs #9246
Comments
It's too bad there aren't protected namespaces on PyPI. I suggest we add a field |
Is it worth temporarily removing the stubs from typeshed for now, as a stopgap measure? |
Btw I would propose to revert that PR for now, so that automatic uploader will work. But this is probably not urgent, I guess people can wait a day or two without stub updates (or we can use manual upload action if something is urgently needed). (also cc @JukkaL for visibility) |
I think mypy has a whitelist of "known" type packages to prevent just such a scenario. |
This allows us to override the generated and uploaded distribution name, e.g. if the name is already in use. Cf. python/typeshed#9246
I've created a PR for the stub_uploader supporting a "stub_distribution" field if we decide to go that way: typeshed-internal/stub_uploader#76. |
Happy to do whatever is needed to resolve this. Sorry for stepping on your namesake in the first place |
I definitely don't think this is your fault, @qubidt :) I don't think there's any reason for typeshed to have any intrinsic "right" to |
I do agree with @ilevkivskyi that we should work out how to collaborate, though, if that's something you'd be interested in @qubidt. It seems silly to have two different sets of stubs for It feels slightly unfair to ask, since you "got there first", but would you be interested in contributing your stubs to typeshed, @qubidt? It looks like you have a lot more "annotation coverage" than us at the moment over at https://github.com/qubidt/types-pika, but I think our CI checks are probably slightly more robust. |
yes, absolutely. although I'm not exactly how to proceed. should I try to merge the annotations from qubidt/types-pika into typeshed and create a PR for it? |
Yes, that would be perfect! Thank you! |
To update on the current status now: typeshed's automated stub uploader is now green again, due to the following trio of PRs by @srittau:
This takes the pressure off, giving us more time to work on consolidating stubs for pika without worrying about the stub uploader constantly failing :) |
The stubs in typeshed are now uploaded to PyPI under the name |
See for example https://github.com/typeshed-internal/stub_uploader/actions/runs/3521987995/jobs/5904439698
PyPI: https://pypi.org/project/types-pika/
Ref #9200
cc @srittau @JelleZijlstra @qubidt
The text was updated successfully, but these errors were encountered: