Skip to content
This repository has been archived by the owner on Mar 11, 2021. It is now read-only.

WIP Upload coverage to coveralls.io #238

Closed
wants to merge 4 commits into from
Closed

WIP Upload coverage to coveralls.io #238

wants to merge 4 commits into from

Conversation

kwk
Copy link
Collaborator

@kwk kwk commented Sep 9, 2016

This change is Reviewable

@kwk kwk added the 💣 test label Sep 9, 2016
@kwk kwk self-assigned this Sep 9, 2016
## Uploads the coverage to coveralls.io
coveralls: $(GOVERALLS_BIN)
cp $(TMP_PATH)/coverage.mode-* profile.cov
$(GOVERALLS_BIN) -coverprofile=profile.cov -repotoken=TcANY144GHGlWbepmrjNV4TAn9Hmsm23t -service=jenkins
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we going to keep the token in repo?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No worries @baijum This is just to test if we want such a service or not. I'll post the results on almighty-public.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kwk Ah. Ok.

I have used that service sometimes back for one of my personal Go project:
https://coveralls.io/github/baijum/usfm?branch=master
It was helpful for me to see the coverage after every push.

@aslakknutsen
Copy link
Contributor

@kwk This is no longer relevant?

@kwk
Copy link
Collaborator Author

kwk commented Sep 24, 2016

@aslakknutsen Yes, I think this is no longer relevant. Because we now have codecov.

@kwk kwk closed this Sep 24, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants