Upload coverage to coveralls.io #1189

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@msabramo
Contributor

msabramo commented Nov 15, 2013

This PR adds support for uploading coverage information to coveralls.io

Sample coveralls.io coverage report:

https://coveralls.io/builds/320023

@msabramo

This comment has been minimized.

Show comment Hide comment
@mmerickel

This comment has been minimized.

Show comment Hide comment
@mmerickel

mmerickel Nov 15, 2013

Owner

I haven't seen coveralls.io before - what advantages does it offer over our travis output?

Owner

mmerickel commented Nov 15, 2013

I haven't seen coveralls.io before - what advantages does it offer over our travis output?

@bertjwregeer

This comment has been minimized.

Show comment Hide comment
@bertjwregeer

bertjwregeer Nov 15, 2013

Owner

The only thing I have noticed is that coveralls spams the hell out of pull requests as changes come in, which is kinda annoying at times.

Owner

bertjwregeer commented Nov 15, 2013

The only thing I have noticed is that coveralls spams the hell out of pull requests as changes come in, which is kinda annoying at times.

@msabramo

This comment has been minimized.

Show comment Hide comment
@msabramo

msabramo Nov 24, 2013

Contributor

Well it shows coverage changes over time and I'm not sure but I think you can fail PRs that reduce coverage.

You can certainly see that the coverage went down and fail it manually; not sure if there is a way to do it automatically.

Contributor

msabramo commented Nov 24, 2013

Well it shows coverage changes over time and I'm not sure but I think you can fail PRs that reduce coverage.

You can certainly see that the coverage went down and fail it manually; not sure if there is a way to do it automatically.

@mmerickel

This comment has been minimized.

Show comment Hide comment
@mmerickel

mmerickel May 12, 2014

Owner

I'm gonna close this. Travis already yells at us when we drop coverage and this is just one more service to keep track of.

Owner

mmerickel commented May 12, 2014

I'm gonna close this. Travis already yells at us when we drop coverage and this is just one more service to keep track of.

@mmerickel mmerickel closed this May 12, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment