Skip to content

Conversation

@gaborbernat
Copy link
Contributor

No description provided.

gaborbernat and others added 3 commits December 6, 2017 20:49
- tox -e py27,py36,coverage allows you to merge and report coverage
files locally too
- tox without args will run all existing python tests and do coverage
report
- codecov is uploaded via a tox env (not ran by default)
@pganssle pganssle merged commit 284dff7 into dateutil:master Dec 6, 2017
@pganssle
Copy link
Member

pganssle commented Dec 6, 2017

@gaborbernat Hrm.. This has a similar problem to one of my earlier configurations - tests are not covered. I thought we were going to set it up so that the regression was that we're not doing changedir anymore?

I'll figure it out later I guess.

@gaborbernat
Copy link
Contributor Author

I explicitly excluded tests from coverage, I can enable it if you want?

@pganssle
Copy link
Member

pganssle commented Dec 7, 2017

@gaborbernat Yes, I've caught several bugs when I realized that parts of my test suite was not getting executed.

@gaborbernat
Copy link
Contributor Author

added back via config flag in tox.ini in #568 (yeah was lazy to create another PR for it, sorry)

@pganssle pganssle added this to the 2.7.0 milestone Dec 7, 2017
@pganssle pganssle mentioned this pull request Mar 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants