FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

move CI to tox by gaborbernat · Pull Request #549 · dateutil/dateutil · GitHub

/ dateutil Public

move CI to tox - #549

Merged
pganssle merged 3 commits into
dateutil:masterfrom
gaborbernat:master
Dec 6, 2017
ConversationCommits (3)ChecksFiles changed
Merged

move CI to tox#549
pganssle merged 3 commits into
dateutil:masterfrom
gaborbernat:master

Conversation

gaborbernat commented Dec 6, 2017

Copy link
Copy Markdown
Contributor

No description provided.

gaborbernat and others added 3 commits December 6, 2017 20:49
make the CI to use tox
5199069 
- 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)
update Author.md with the contribution
255f7d7
Merge branch 'master' into master
6a6f068
pganssle merged commit 284dff7 into dateutil:master Dec 6, 2017
gaborbernat mentioned this pull request Dec 6, 2017
Closed

pganssle commented Dec 6, 2017

Copy link
Copy Markdown
Member

@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 commented Dec 7, 2017

Copy link
Copy Markdown
Contributor Author

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

pganssle commented Dec 7, 2017

Copy link
Copy Markdown
Member

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

gaborbernat commented Dec 7, 2017

Copy link
Copy Markdown
Contributor Author

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

pganssle added this to the 2.7.0 milestone Dec 7, 2017
pganssle mentioned this pull request Mar 11, 2018
Closed
This file contains hidden or 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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

build tests

Projects

None yet

Milestone

2.7.0

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL