-
Notifications
You must be signed in to change notification settings - Fork 90
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 support for Go/Golang coverage reports #303
Comments
@adrienjoly Thanks for submitting an issue! |
Unfortunately, I believe that Go uses its own format, for coverage reports...
You can find some pointers on how to convert to more standard formats, on this stackoverflow page: https://stackoverflow.com/questions/31413281/golang-coverprofile-output-format/39453143
|
I have https://facebook.github.io/watchman/ configured to convert the go coverage to lcov format using https://github.com/jandelgado/gcov2lcov
This works great and the |
@munnik Thanks for that work around, it should help users in the meantime until we get around to implementing the go coverage format natively in the extension! cc @adrienjoly |
For anyone coming to this now, who isn't that familiar with
And since I had
1: https://facebook.github.io/watchman/docs/watchman-make.html |
@williammartin Thanks for the extra info 👍🏻, I am sure someone will find the above valuable. |
Is your feature request related to a problem? Please describe.
Go/Golang coverage reports do not seem directly supported by this extension.
Describe the solution you'd like
It would be awesome if the extension could understand the report generated by
go test
, out of the box.Describe alternatives you've considered
I tested another extension that was supposed to display the coverage of .go files, but it failed with an error message at startup.
Additional context
Here's a workaround that I used successfully to make Vscode Coverage Gutters display the coverage of my go files, run the following command:
... then, re-activate the "watch" if necessary. You should get something like that:
The text was updated successfully, but these errors were encountered: