Skip to content
This repository was archived by the owner on Oct 18, 2024. It is now read-only.

Add note about Codecov integration - #75

Merged
axw merged 2 commits into
axw:masterfrom
stevepeak:patch-1
Mar 26, 2016
Merged

Add note about Codecov integration#75
axw merged 2 commits into
axw:masterfrom
stevepeak:patch-1

Conversation

@stevepeak

Copy link
Copy Markdown

Waiting on AlekSi/gocov-xml#6

Will adjust the Learn more to link to the changes proposed above. Thank you!

@axw

axw commented Mar 21, 2016

Copy link
Copy Markdown
Owner

@stevepeak Happy to add a link once the gocov-xml change is made.

It might be worth changing the section title to "Related tools/services", and having Codecov on its own line underneath gocov-xml, with mention of gocov-xml as a requirement. Please keep it as a simple link and brief description, and drop the "Learn More". Interested parties can click through the link to do that.

@stevepeak

Copy link
Copy Markdown
Author

How does this look? Thank you @axw 👍

@axw

axw commented Mar 25, 2016

Copy link
Copy Markdown
Owner

@stevepeak Looks fine, thanks. Please ping when gocov-xml is ready.

@stevepeak

Copy link
Copy Markdown
Author

We are all set. Cobertura has been an acceptable format since birth :) Thank you!

@axw
axw merged commit dbb7f55 into axw:master Mar 26, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants