Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

RF: coveralls (not used/relied on really) -> codecov#522

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
Byron merged 1 commit intogitpython-developers:masterfromyarikoptic:enh-codecov
Oct 9, 2016

Conversation

yarikoptic
Copy link
Contributor

@yarikopticyarikoptic commentedOct 2, 2016•
edited
Loading

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. athttps://github.com/datalad/datalad/pull/969/files

@yarikopticyarikoptic changed the titleRF: coveralls (not used/relied on really) -> codecoRF: coveralls (not used/relied on really) -> codecovOct 2, 2016
codecov in our (datalad, etc) experience provides a better service,great support, and super-nice intergration with chromium and firefoxfor reviewing coverage of pull requests.  In light of the@with_rw_directory fiasco detected/fixed ingitpython-developers#521 I would stronglyrecommend to (re-)enable and use coverage reports
@codecov-io
Copy link

codecov-io commentedOct 2, 2016•
edited
Loading

Current coverage is 86.11% (diff: 100%)

No coverage report found formaster atdf5c1cb.

Powered byCodecov. Last updatedf5c1cb...94ca83c

@ankostis
Copy link
Contributor

That seems cool.@Byron has to decide and implement the change.

@yarikoptic
Copy link
ContributorAuthor

Please let me know if I could be of additional help in persuasion :-)

@ankostisankostisforce-pushed themaster branch 2 times, most recently from391a767 to8a2f7dcCompareOctober 4, 2016 00:12
@Byron
Copy link
Member

No persuasion required :) ! I just want to make sure we get what we need to keep GitPython up and running !

@ByronByron merged commit6f6c697 intogitpython-developers:masterOct 9, 2016
@yarikopticyarikoptic deleted the enh-codecov branchOctober 15, 2016 05:27
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@yarikoptic@codecov-io@ankostis@Byron
[8]ページ先頭
©2009-2025 Movatter.jp