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

cmd/test-coverage: implement a new command assessing code coverage on a whole repo #5

Open
sbinet opened this issue Jul 24, 2018 · 4 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@sbinet
Copy link
Member

sbinet commented Jul 24, 2018

right now, this is done with a shell script:

that shell script is working around the fact that go test -coverprofile=profile.out can only work for a single package (and not for a bunch of packages, like go test ./... does)

rewriting that script in Go could be helpful (and be reused by other projects.)

@sbinet sbinet added enhancement New feature or request good first issue Good for newcomers labels Jul 24, 2018
@ahlusar1989
Copy link

@sbinet I'll take a look at this

@ahlusar1989
Copy link

@sbinet Is this something that could be used to traverse each path and amalgamate the profiles:

https://github.com/go-playground/overalls

@sbinet
Copy link
Member Author

sbinet commented Aug 3, 2018

Hum... Yes, probably :)
(Then this issue is probably just about modifying our scripts to use that.)

@ahlusar1989
Copy link

Sounds good @sbinet - on it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants