-
-
Notifications
You must be signed in to change notification settings - Fork 102
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
Tox based tests #263
Tox based tests #263
Conversation
Codecov Report
@@ Coverage Diff @@
## master #263 +/- ##
=======================================
Coverage 99.71% 99.71%
=======================================
Files 4 4
Lines 355 355
=======================================
Hits 354 354
Misses 1 1 Continue to review full report at Codecov.
|
@webknjaz thank you for the PR |
Sure, it's only a demo for now. I understand importance of non-breaking the current flow + I can also help you to figure out some benefits of working via tox for your tasks. |
@asvetlov any updates so far? |
Works fast.
|
I suppose that it's reporter to codecov task is being broken, not coverage collection. Right? I'll fix this by making tox not build a wheel in that env, I guess.
So, when you run tox now it pick up whatever points to BonusI've wired up pytest-watch with pytest-testmon as |
eef1018
to
92c37ae
Compare
mypy failure is unrelated to this PR, it's related to the version change. 0.610 completed without errors, 0.620 errors out. |
coverage issue solved. and I've sped-up a number of auxiliary envs. |
(Circle CI is failing because new image is still in process of building in Docker Hub) |
e8b6ab6
to
7e72d6e
Compare
Including: * AppVeyor * Circle CI * Travis CI
Including: * Travis * AppVeyor * Also moved twine upload to tox * Shippable
7e72d6e
to
3cb6212
Compare
I've merged this assuming that all questions are addressed now. However, I'm always open to cosmetic changes. |
This attempts to make tox a single source of truth for all the test envs (mainly CI, but it wires it into GNU make as well.
I didn't test deployment flow yet, but current results look promising in terms of clearly separating different envs/tasks + there's a nice ~10% speedup in Travis.