-
Notifications
You must be signed in to change notification settings - Fork 67
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
CI/CD publishing to PyPi #43
Comments
Thanks for your proposal Domi. |
Thanks Benoit. I think starting with Linux and CPython builds is already a major step forward, so I took the liberty and created a PR for your review: PR #44 |
Let me know if there are any issues with the PR, or you decided internally in P1 against it. If you want to see the resulting wheels check the artifacts in my fork (scroll to the bottom): |
Thanks Domi. |
Hi Domi, Your PR has been integrated, kudos and thanks for it! |
Hi Benoit,
I have seen you already started with CI/CD publishing of pycrate wheel to pypi. Thank you very much for that! I was wondering if you would be open to do it for this project as well? It would be much more convenient to have the binary wheels on pypi instead of just the source code, so server installation could be simplified (no need to maintain local builds or install gcc and headers onto servers).
I would be happy to contribute a simplified CI/CD yaml file for this project based on what you have in pycrate (https://github.com/P1sec/pycrate/blob/master/.github/workflows/cicd.yml), but it would still need your token to Pypi set in this project, so first I wanted to know if you would be open to support this?
Simplified would mean: rip out everything there is for Windows and Mac (?) because AFAIR pysctp is Linux only, and adjust tests etc.
Thanks,
Domi
The text was updated successfully, but these errors were encountered: