-
Notifications
You must be signed in to change notification settings - Fork 80
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
[MRG] CI: moving python tests from travis to GH actions #1249
Conversation
Codecov Report
@@ Coverage Diff @@
## latest #1249 +/- ##
===========================================
+ Coverage 43.69% 92.87% +49.18%
===========================================
Files 103 76 -27
Lines 9533 5977 -3556
===========================================
+ Hits 4165 5551 +1386
+ Misses 5368 426 -4942
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
c9a4266
to
b903c5e
Compare
Ready for review and merge @ctb |
@@ -3,6 +3,7 @@ name: cibuildwheel | |||
on: | |||
push: | |||
branches: [latest] | |||
tags: v* |
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.
(and this should fix building wheels on new tags, @ctb: try to force push 4.0.0a2
again after this is merged =])
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.
nice!
Turns out we consume the Travis credits too fast and it is blocking our PRs. This PR moves the Python tests from Travis to GitHub Actions.
I made the Python tests on 3.9 required, as well as wheel building on Linux and MacOS.
Travis is still running arm64 and ppcle64 wheels on new tags, but otherwise all tests were transferred to GitHub Actions with this PR.
TODO
Checklist
make test
Did it pass the tests?make coverage
Is the new code covered?without a major version increment. Changing file formats also requires a
major version number increment.
changes were made?