Skip to content

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

Merged
merged 3 commits into from
Dec 22, 2020

Conversation

thomasjpfan
Copy link
Member

@thomasjpfan thomasjpfan commented Dec 22, 2020

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 then pypi.

@thomasjpfan thomasjpfan mentioned this pull request Dec 22, 2020
14 tasks
Copy link
Member

@alfaro96 alfaro96 left a 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.

@thomasjpfan
Copy link
Member Author

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

@glemaitre
Copy link
Member

I already pushed the following to master to upload the wheels for the 0.24.0 release: ae3c48c

@ogrisel
Copy link
Member

ogrisel commented Dec 22, 2020

I would be in favor of merging this PR but it needs to undo the change of ae3c48c.

@alfaro96
Copy link
Member

I would be in favor of merging this PR but it needs to undo the change of ae3c48c.

+1 for this PR too.

Copy link
Member

@ogrisel ogrisel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ogrisel
Copy link
Member

ogrisel commented Dec 22, 2020

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.

@ogrisel ogrisel merged commit aacb1de into scikit-learn:master Dec 22, 2020
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Jan 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants