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

Send test coverage data to coveralls.io #167

Merged
merged 3 commits into from
Mar 9, 2015

Conversation

cichli
Copy link
Member

@cichli cichli commented Mar 8, 2015

I thought this was pretty cool - see what the reports look like here or for a specific file here. It doesn't support partial line coverage (see lemurheavy/coveralls-public#216) so cloverage uses 1x line hit for partial coverage and 2x line hits for complete coverage.

PRs will have an extra status added with information on how test coverage has changed if you enable coveralls for the repo at http://coveralls.io (only needs status API permissions).

@bbatsov
Copy link
Member

bbatsov commented Mar 8, 2015

I'm familiar with coveralls - I use it in various Ruby projects. There's a useful badge that we might add to the README. It shows the current coverage.

@cichli cichli force-pushed the cloverage-coveralls branch from d28892c to 5e916d1 Compare March 9, 2015 18:13
@cichli
Copy link
Member Author

cichli commented Mar 9, 2015

I've added a badge next to the Travis badge in README.md.

bbatsov added a commit that referenced this pull request Mar 9, 2015
Send test coverage data to coveralls.io
@bbatsov bbatsov merged commit 038aed6 into clojure-emacs:master Mar 9, 2015
@bbatsov
Copy link
Member

bbatsov commented Mar 9, 2015

👍

@cichli
Copy link
Member Author

cichli commented Mar 9, 2015

Now to make that number go up :-). If you browse to a file in the report it can't find the source - think you need to tell coveralls the the paths are relative to /src/ on that page.

@cichli cichli deleted the cloverage-coveralls branch March 9, 2015 18:42
@bbatsov
Copy link
Member

bbatsov commented Mar 9, 2015

Done.

On 9 March 2015 at 20:42, Michael Griffiths [email protected]
wrote:

Now to make that number go down :-). If you browse to a file
https://coveralls.io/files/530587217 in the report it can't find the
source - think you need to tell coveralls the the paths are relative to
/src/ on that page.


Reply to this email directly or view it on GitHub
#167 (comment)
.

Best Regards,
Bozhidar Batsov

http://www.batsov.com

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants