-
Notifications
You must be signed in to change notification settings - Fork 32
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
Publish to pypi.org #30
Comments
Based on skimming the documentation it looks straightforward. One decision that would need to be made is whether to enable the |
I've never done this myself, where is the documentation? Presumably this could be done in a CI step so that it's the same process every time, not done on a developer's machine. |
https://packaging.python.org/tutorials/packaging-projects/ It's not immediately clear to me if non-human accounts are permitted. |
@ricea and @foolip over on web-platform-tests/wpt#35978 we tried to release a new version of wptserve on PyPI to unblock a couple of other projects depending on a newer version. But sadly it fails because this package is still not published on PyPI. Do you think that this could be done? For now we probably can use the link to the git repository but it would be nicer to see a release on PyPI itself. |
Also if it should be published as |
If you have experiment with packaging for PyPI your help would be greatly appreciated. |
The first thing that should be clear is definitely the name. I assume that |
Oh, the clash with the "other" pywebsocket is unfortunate. I'd rather keep the name |
The PR for renaming the module has just been merged. Now we have two questions to follow-up:
@ricea maybe you can give some feedback? Thanks! |
I'm investigating the CI question. I agree with your opinion that we should bump the version to 4.0.0 as this was a breaking change. I was planning to use 4.0.0 for the first version without Python 2 support, but we can make that 5.0.0 😃 |
Is there more work to do to get rid of Python 2 support beside removing the usage of |
AFAIK removing usage of |
Sorry for the delay. I tagged release v4.0.0. Please check if it looks okay. |
Thanks Adam. I pulled and run all the tests locally. All of them pass, so it looks fine to me. I only filed a new issue for a deprecation warning. |
Hi @ricea. Do you have an update regarding the release for PyPI? I would appreciate if you could upload it. Thanks. |
A release on PyPI would not only help us with web-platform tests but also would unblock several other projects that are waiting for an official release of this tool. @foolip maybe you know who else could help to get the package released on PyPI? Thanks. |
Is there anyone else who could help with this issue and the release on PyPI? |
I've asked @quasi-mod to take it over. |
This patch adds publish.yml to enable the workflow to publish to PyPi.
This patch adds a trigger event to publish.yml and specifies the release event to when we will publish to PyPi.
pywebsocket3 is officially on PyPi! I have set GitHub actions to make new PyPi releases when a release is published on GitHub. Thanks so much for helping out :) |
Thanks a lot for making it available on PyPI! Maybe some more details could be added to the readme to give a better description for the package on Pypi? |
pywebsocket3 is vendored into web-platform-tests, and it looks like there is no released package which could be used to avoid the copying. Can pywebsocket3 be released on pypi.org to make this possible? (Not imminent, but eventually.)
The text was updated successfully, but these errors were encountered: