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

Upload code coverage to codecov.io #995

Merged
merged 8 commits into from
Jan 6, 2023

Conversation

martincostello
Copy link
Member

@martincostello martincostello commented Jan 6, 2023

The issue or feature being addressed

  • Upload code coverage reports to codecov.io, generated with ReportGenerator in Cobertura format.
  • Update NuGet packages to their latest versions.
  • Remove net461 as a test target.
  • Update Cake and associated tools to their latest versions.
  • Add GitHubActionsTestLogger so test results are shown in workflow logs.
  • Fix GitVersion not working when run in GitHub Actions.

Contributes to #980.

Still WIP as I'm having trouble getting ReportGenerator to actually read the output from coverlet.

Details on the issue fix or feature implementation

Confirm the following

  • I started this PR by branching from the head of the default branch
  • I have targeted the PR to merge into the default branch
  • I have included unit tests for the issue/feature
  • I have successfully run a local build

- Upload code coverage reports to codecov.io, generated with ReportGenerator in Cobertura format.
- Update NuGet packages to their latest versions.
- Remove `net461` as a test target.
Add a code coverage badge to the README.
Add GitHubActionsTestLogger for builds in GitHub Actions.
Contributes to App-vNext#980.
Update Cake to version 3, as well as all the related dependencies.
Remove leftover Cake task for AppVeyor.
Fix the Cake build so that the correct version properties are set for the binary and NuGet package.
Also modernises some terminology for .NET (Core).
Output from coverlet in cobertura format so that ReportGenerator can process it.
@martincostello martincostello marked this pull request as ready for review January 6, 2023 18:28
@martincostello martincostello merged commit d3c4e1d into App-vNext:main Jan 6, 2023
@martincostello martincostello deleted the add-code-coverage branch January 6, 2023 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant