Skip to content

misc: update wheel building and release scripts - #9570

Merged
hauntsaninja merged 4 commits into
python:masterfrom
hauntsaninja:whl
Jan 7, 2021
Merged

hauntsaninja merged 4 commits into
python:masterfrom
hauntsaninja:whl

Conversation

@hauntsaninja

@hauntsaninja hauntsaninja commented Oct 9, 2020 •

Copy link
Copy Markdown
Collaborator

This goes together with mypyc/mypy_mypyc-wheels#11 to resolve #9536

download-mypyc-wheels is now subsumed by upload-pypi, which uses the Github API instead of hardcoding filenames. This also means you won't need to make changes like #9559 If this feels fragile, let me know and I can add back a check against a hardcoded list of names (note CI won't make a release if one of the builds fails).

test_installed_version is part of mypyc_mypy-wheels and some of what it does is taken care of by cibuildwheel

upload-pypi can be made a lot simpler, since CI now builds the pure Python wheel and sdist. Since it's no longer building things, I assume we can be less picky about what Python we use. The version check now runs against the sdist, which is nice.

Some of this was a little hard to test end to end, but if there are kinks it shouldn't be too hard to work them out.

hauntsaninja added 3 commits October 9, 2020 14:10
CI now builds the pure Python wheel and sdist.
Use the Github API to pull down all assets associated with the CI build.
Because this script now does very little, it's a lot simpler.
Since it's no longer building things, I assume we can be less picky
about what Python we use.
The version check now runs against the sdist, which is nice.
download-mypyc-wheels is now subsumed by upload-pypi, which uses the
Github API instead of hardcoding filenames.

test_installed_version is part of mypyc_mypy-wheels and some of what it
does is taken care of by cibuildwheel

@msullivan msullivan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This seems pretty reasonable to me and we should move forward with it. We should coordinate merging it so I can bother you about any "fine-tuning" that needs to be done.

@hauntsaninja hauntsaninja mentioned this pull request Nov 26, 2020
5 of 6 tasks
@hauntsaninja
hauntsaninja merged commit 1f5115b into python:master Jan 7, 2021
@hauntsaninja
hauntsaninja deleted the whl branch January 7, 2021 03:18
ilevkivskyi pushed a commit that referenced this pull request Jan 19, 2021
* new upload pypi script

CI now builds the pure Python wheel and sdist.
Use the Github API to pull down all assets associated with the CI build.
Because this script now does very little, it's a lot simpler.
Since it's no longer building things, I assume we can be less picky
about what Python we use.
The version check now runs against the sdist, which is nice.

* update trigger wheel build for new wheel building

* remove more scripts

download-mypyc-wheels is now subsumed by upload-pypi, which uses the
Github API instead of hardcoding filenames.

test_installed_version is part of mypyc_mypy-wheels and some of what it
does is taken care of by cibuildwheel

* fix repo :-)

Co-authored-by: hauntsaninja <>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Speed up mypy wheel builds

2 participants