Skip to content
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

travis-ci: upload coverage to CodeCov #751

Merged
merged 2 commits into from Aug 11, 2016

Conversation

Projects
None yet
4 participants
@grondo
Copy link
Contributor

grondo commented Aug 10, 2016

Add codecov.io for coverage results in addition to coveralls.io.

CodeCov seems to have a much better interface than Coveralls, and includes "coverage diffs" in PRs which I think will come in very useful.

Check out basic example here

Will rebase on #749 once that is separately merged.

@grondo grondo added the review label Aug 10, 2016

@coveralls

This comment has been minimized.

Copy link

coveralls commented Aug 11, 2016

Coverage Status

Coverage decreased (-0.05%) to 75.025% when pulling 4d388c2 on grondo:codecov into 1fd19b2 on flux-framework:master.

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Aug 11, 2016

Current coverage is 74.67% (diff: 100%)

No coverage report found for master at 3ab9ec8.

Powered by Codecov. Last update 3ab9ec8...325415d

@garlick

This comment has been minimized.

Copy link
Member

garlick commented Aug 11, 2016

Agreed the interface seems nicer.

@grondo

This comment has been minimized.

Copy link
Contributor Author

grondo commented Aug 11, 2016

I'll rebase this once I get back from 6th grade orientation!

On Aug 11, 2016 8:40 AM, "Jim Garlick" notifications@github.com wrote:

Agreed the interface seems nicer.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#751 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtSUk-OGGH90gVp1Klpq7XrADK52AW3ks5qe0JwgaJpZM4JhrsW
.

grondo added some commits Aug 10, 2016

travis-ci: send coverage results to codecov.io
In addition to coveralls, send code coverage results to
codecov.io, which seems to have more coverage monitoring
features, including coverage diffs.

@grondo grondo force-pushed the grondo:codecov branch from 4d388c2 to 325415d Aug 11, 2016

@grondo

This comment has been minimized.

Copy link
Contributor Author

grondo commented Aug 11, 2016

Ok, rebased on current master.

@coveralls

This comment has been minimized.

Copy link

coveralls commented Aug 11, 2016

Coverage Status

Coverage decreased (-0.008%) to 74.99% when pulling 325415d on grondo:codecov into 3ab9ec8 on flux-framework:master.

@garlick garlick merged commit cb751ac into flux-framework:master Aug 11, 2016

3 checks passed

codecov/patch Coverage not affected.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage decreased (-0.008%) to 74.99%
Details

@garlick garlick removed the review label Aug 11, 2016

@grondo grondo deleted the grondo:codecov branch Aug 15, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.