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

add coverage badge once multiple packages can be tested #134

Open
stapelberg opened this issue Jun 26, 2016 · 0 comments
Open

add coverage badge once multiple packages can be tested #134

stapelberg opened this issue Jun 26, 2016 · 0 comments
Labels

Comments

@stapelberg
Copy link
Contributor

Commit 7b75a3c adds coveralls integration, see https://coveralls.io/github/robustirc/robustirc for the results. This however only runs tests in github.com/robustirc/robustirc, not github.com/robustirc/robustirc/...:

$ goveralls -package=github.com/robustirc/robustirc/... 
exit status 1: cannot use test profile flag with multiple packages

IIUC, this is because of a fundamental limitation of the cover command, see golang/go#6909

Once that issue is fixed, we should tell goveralls to run all our tests and then include the coverage badge. Until then, I don’t see much value in exposing a number which massively mis-represents our code/coverage.

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

No branches or pull requests

1 participant