Repository navigation
RF: coveralls (not used/relied on really) -> codecov - #522
Merged
Byron merged 1 commit intoOct 9, 2016
Merged
Byron merged 1 commit into
Byron merged 1 commit into
Conversation
Copy link
Copy Markdown
Contributor
Sorry, something went wrong.
codecov in our (datalad, etc) experience provides a better service, great support, and super-nice intergration with chromium and firefox for reviewing coverage of pull requests. In light of the @with_rw_directory fiasco detected/fixed in gitpython-developers#521 I would strongly recommend to (re-)enable and use coverage reports
yarikoptic
force-pushed
the
enh-codecov
branch
from
18caa95 to
94ca83c
Compare
October 2, 2016 15:00
Copy link
Copy Markdown
Current coverage is 86.11% (diff: 100%)
|
Sorry, something went wrong.
Copy link
Copy Markdown
Contributor
|
That seems cool. @Byron has to decide and implement the change. |
Sorry, something went wrong.
Copy link
Copy Markdown
Contributor
Author
|
Please let me know if I could be of additional help in persuasion :-) |
Sorry, something went wrong.
ankostis
force-pushed
the
master
branch
2 times, most recently
from
391a767 to
8a2f7dc
Compare
October 4, 2016 00:12
Copy link
Copy Markdown
Member
|
No persuasion required :) ! I just want to make sure we get what we need to keep GitPython up and running ! |
Sorry, something went wrong.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
codecov in our (datalad, etc) experience provides a better service,
great support, and super-nice intergration with chromium and firefox
for reviewing coverage of pull requests. In light of the
@with_rw_directory fiasco detected/fixed in #521 I would strongly
recommend to (re-)enable and use coverage reports
edit 1: I think that if you accept this PR codecov might automagically enable itself on this repo, but I could be wrong and someone with authority might need to login into codecov via github and enable
coverage as seen from my clone: https://codecov.io/gh/yarikoptic/GitPython/commit/94ca83c6b6f49bb1244569030ce7989d4e01495c
and trust me integration with browsers is great! ha -- actually basic (since there were no reports against master -- can't show the difference) report already appeared here (magic!)
If you want to see it in action -- install codecov browser plugin and look e.g. at https://github.com/datalad/datalad/pull/969/files