-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
CI Publish to Pypi workflow for aarch64 wheels #19060
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @thomasjpfan!
LGTM.
I have been testing this script locally by running: rm -rf dist
python -m wheelhouse_uploader fetch \
--version 0.24.0 \
--local-folder dist/ \
scikit-learn \
https://pypi.anaconda.org/scikit-learn-wheels-staging/simple/scikit-learn/
python build_tools/github/check_wheels.py |
I already pushed the following to master to upload the wheels for the 0.24.0 release: ae3c48c |
I would be in favor of merging this PR but it needs to undo the change of ae3c48c. |
+1 for this PR too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Let's wait for the 0.24.0 doc build on circle CI to complete before merging a PR to master: https://app.circleci.com/pipelines/github/scikit-learn/scikit-learn This is to avoid a git race condition between circle ci runs on the 0.24.X vs master branch when deploying the new website. |
Reference Issues/PRs
Related to #19058
What does this implement/fix? Explain your changes.
Updates the "Publish to Pypi" workflow so that it counts the aarch64 builds from the travis config.
This only needs to be merged with master. Then we can trigger the workflow to upload to
testpypi
and thenpypi
.