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

Enable code coverage metrics #540

Closed
ctron opened this issue Mar 19, 2018 · 3 comments
Closed

Enable code coverage metrics #540

ctron opened this issue Mar 19, 2018 · 3 comments
Assignees
Labels
CQ Waiting for CQ approval
Milestone

Comments

@ctron
Copy link
Contributor

ctron commented Mar 19, 2018

I think it would be good if we enable code coverage metrics. My suggestion would be to use jacoco with maven for the coverage and codecov.io for visualizing the numbers. Of course those are only suggestions and I am open of alternatives.

@ctron ctron self-assigned this Mar 19, 2018
@ctron
Copy link
Contributor Author

ctron commented Mar 19, 2018

Also as @sysexcontrol mentioned on the maillinglist. The build should not be failed due to "bad metrics".

ctron added a commit to ctron/hono that referenced this issue Mar 19, 2018
This will enable code coverage metrics using jacoco and visualization
using codecov.io.
ctron added a commit to ctron/hono that referenced this issue Mar 19, 2018
This will enable code coverage metrics using jacoco and visualization
using codecov.io.
@sophokles73
Copy link
Contributor

@ctron can we (you) close this issue?

@ctron
Copy link
Contributor Author

ctron commented May 29, 2018

I will check …

@ctron ctron added this to the 0.8 milestone Aug 23, 2018
ctron added a commit to ctron/hono that referenced this issue Aug 27, 2018
This will enable code coverage metrics using jacoco and visualization
using codecov.io.
@ctron ctron added the CQ Waiting for CQ approval label Aug 27, 2018
ctron added a commit that referenced this issue Sep 4, 2018
This will enable code coverage metrics using jacoco and visualization
using codecov.io.
@ctron ctron closed this as completed Sep 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CQ Waiting for CQ approval
Projects
None yet
Development

No branches or pull requests

2 participants