Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

badge: Remove codacy coverage badge #144

Merged
merged 1 commit into from
Jul 6, 2017
Merged

badge: Remove codacy coverage badge #144

merged 1 commit into from
Jul 6, 2017

Conversation

gaocegege
Copy link
Member

@gaocegege gaocegege commented Jul 6, 2017

The coverage report has bugs caused by codacy, so remove the badge.

ref #103 and codacy/codacy-coverage-reporter#56

Signed-off-by: Ce Gao ce.gao@outlook.com

Signed-off-by: Ce Gao <ce.gao@outlook.com>
@gaocegege gaocegege merged commit 2ba8675 into master Jul 6, 2017
@gaocegege gaocegege deleted the badge/remove branch July 6, 2017 06:57
@jeremydouglass
Copy link
Member

Remind me -- are you using something for a coverage report right now outside github?

@gaocegege
Copy link
Member Author

I am using jacoco, but the report is not visible in web.

@gaocegege gaocegege modified the milestone: Evaluation 2 Jul 12, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants