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

chore: add code coverage reporting #243

Merged
merged 1 commit into from
Feb 12, 2021
Merged

Conversation

jawnsy
Copy link
Contributor

@jawnsy jawnsy commented Feb 12, 2021

  • Update actions to latest versions
  • Update golangci-lint to latest version, 1.36
  • Pin Ubuntu to the latest, version 20.04
  • Add coverage profiling test and send results to Coveralls

@shortcut-integration
Copy link

This pull request has been linked to Clubhouse Story #7019: Coverage reporting in pull requests.

@jawnsy jawnsy force-pushed the jawnsy/ch7019/goveralls branch from 3db1dfa to 8091c42 Compare February 12, 2021 05:47
@jawnsy jawnsy changed the title test: Add code coverage reporting chore: add code coverage reporting Feb 12, 2021
@jawnsy jawnsy self-assigned this Feb 12, 2021
@jawnsy jawnsy requested review from cmoog and tychoish February 12, 2021 05:50
* Update actions to latest versions
* Update golangci-lint to latest version, 1.36
* Pin Ubuntu to the latest, version 20.04
* Add coverage profiling test and send results to Coveralls
@jawnsy jawnsy force-pushed the jawnsy/ch7019/goveralls branch from 8091c42 to a185b80 Compare February 12, 2021 05:51
@jawnsy jawnsy marked this pull request as ready for review February 12, 2021 05:55
Copy link

@tychoish tychoish left a comment

Choose a reason for hiding this comment

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

does this require credentials?

@jawnsy
Copy link
Contributor Author

jawnsy commented Feb 12, 2021

@tychoish Nah it does some sort of GitHub SSO, so uses the GITHUB_TOKEN to upload. You should be able to access the reports using your GitHub login also

@jawnsy jawnsy merged commit ca68fdb into master Feb 12, 2021
@jawnsy jawnsy deleted the jawnsy/ch7019/goveralls branch February 12, 2021 15:59
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.

3 participants