This repository has been archived by the owner on Jul 17, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 16
Use cibuildwheel to produce wheels #42
Closed
Closed
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
9dffe1b
Experiment with minimal cibuildwheel config
madig 601521a
Add testing
madig 319f719
AppVeyor: backslash path, Travis: Yaml syntax
madig db88fcd
AppVeyor: more syntax
madig 6edbc6a
Try coverage testing
madig 0a477c9
Look in tests/ for coverage data
madig f73d293
Just tox
madig 69736e1
Add venv and .mypy_* to gitignore
madig 47166c6
tox more correctly
madig 28895f1
Fix ERROR: toxini file 'tox.ini' not found
miguelsousa 4ded39a
Try tox --installpkg $wheel
miguelsousa f56bb1e
Move --installpkg to CIBW_TEST_COMMAND
miguelsousa 179de72
Use wheelhouse
miguelsousa a1f00e5
Run the tests first
miguelsousa 06dcb42
Skip the codecov upload for now [skip appveyor]
miguelsousa 869fc96
Try with a different path [skip appveyor]
miguelsousa File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
we don't want to run tox ourselves again, after cibuildwheel. We want the
CIBW_TEST_COMMAND
to call tox with the --installpkg option...cibuildwheel builds the wheel, then installs it and runs the given test command. We want tox to take care of configuring the test environment, but we want to make it use a pre-build package (the wheel that we just built with cibuildwheel), instead of rebuilding the module from a sdist (which is its default behavior).
I need to go now, I'll have a look again tomorrow.