-
Notifications
You must be signed in to change notification settings - Fork 299
Use tox to run test in travis #440
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
This way we can harmonize tox and travis configuration
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!
Codecov Report
@@ Coverage Diff @@ ## master #440 +/- ## ========================================= + Coverage 91.95% 92.6% +0.65% ========================================= Files 57 54 -3 Lines 2994 2812 -182 ========================================= - Hits 2753 2604 -149 + Misses 241 208 -33
Continue to review full report at Codecov.
|
97b42d6 to
7d7d59d
Compare
@mblayman I would have been too lucky to get it right in the first run. Coverage was not working anymore and is now fixed. Could you have a look again? Thanks.
7d7d59d to
0f01240
Compare
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.
Still looks good to me. Thanks for doing this one. I think it will help the tox file stay up-to-date. 👍
This way we can harmonize tox and travis configuration
Fixes #391
Description of the Change
Checklist
CHANGELOG.mdAUTHORS