switching from coverals to codecov #764

Merged
merged 3 commits into from Jan 5, 2017

Conversation

Projects
None yet
3 participants
Contributor

lbergelson commented Nov 29, 2016

codecov seems to be more accurate since it incorporates a notion of "partially covered lines"
it also makes reviewing pr's easier since coverage can be shown inline during reviews with a browser plugin

@lbergelson lbergelson switching from coverals to codecov
codecov seems to be more accurate since it incorporates a notion of "partially covered lines"
it also makes reviewing pr's easier since coverage can be shown inline during reviews with a browser plugin
57bf8df
Contributor

lbergelson commented Nov 29, 2016

don't merge this, it isn't capturing the sra test coverage yet

lbergelson referenced this pull request Dec 5, 2016

Merged

updating the gradle version from 2.13 to 3.2.1 #727

2 of 5 tasks complete

lbergelson was assigned by droazen Dec 13, 2016

@lbergelson lbergelson seeing if running jacoco test report AFTER sra tests will include them
bddbf6a

codecov-io commented Jan 4, 2017 edited

Current coverage is 63.981% (diff: 100%)

Sunburst

No coverage report found for master at 6e4e875.

Powered by Codecov. Last update 6e4e875...b7d8ba3

Contributor

yfarjoun commented Jan 4, 2017

Contributor

lbergelson commented Jan 4, 2017

@yfarjoun Have you seen the browser plugin that lets you see which lines of a pull request are covered and which aren't?

@lbergelson lbergelson updating readme and adding yml
b7d8ba3
Contributor

lbergelson commented Jan 4, 2017

@yfarjoun I'm not certain but I suspect that if the SRA tests started working the coverage would be included now. Could you review this?

@lbergelson lbergelson assigned yfarjoun and unassigned lbergelson Jan 4, 2017

@lbergelson lbergelson merged commit ffca259 into master Jan 5, 2017

3 checks passed

codecov/patch Coverage not affected.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment