Upload coverage report to coveralls.io #367

nwithan8 · 2022-11-03T22:04:29Z

Description

Temporarily enabled for all pushes (not just master merges), report uploaded correctly: https://coveralls.io/github/EasyPost/easypost-csharp

Please select the option(s) that are relevant to this PR.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Improvement (fixing a typo, updating readme, renaming a variable name, etc)

- Add Coveralls badge to README.md - Only run coverage on pushes to master

Justintime50

Tiny thing, otherwise good!

Makefile

- Upload coverage report to coveralls.io

3456bd0

nwithan8 requested a review from a team as a code owner November 3, 2022 22:04

- Fix env var for coveralls

f4e0267

nwithan8 marked this pull request as draft November 3, 2022 22:09

nwithan8 added 2 commits November 3, 2022 16:20

- Use coveralls GitHub Action rather than coveralls dotnet tool

ff5c597

- Remove HTML report upload

0a9d21e

- Add Coveralls badge to README.md - Only run coverage on pushes to master

nwithan8 marked this pull request as ready for review November 3, 2022 22:29

Justintime50 reviewed Nov 3, 2022

View reviewed changes

Makefile Outdated Show resolved Hide resolved

- Remove coveralls Makefile option

d3e77d6

nwithan8 requested a review from Justintime50 November 3, 2022 22:33

Justintime50 approved these changes Nov 3, 2022

View reviewed changes

jchen293 approved these changes Nov 4, 2022

View reviewed changes

nwithan8 merged commit 368e9d8 into master Nov 4, 2022

nwithan8 deleted the coveralls branch November 4, 2022 16:28