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

Switch to codecov #11

Merged
merged 33 commits into from
Nov 13, 2023
Merged

Switch to codecov #11

merged 33 commits into from
Nov 13, 2023

Conversation

FelixS90
Copy link
Collaborator

@FelixS90 FelixS90 commented Nov 9, 2023

Description

Use codecov to have fullstack code coverage support in PR, dashboard in one solution
No need for any other third party actions

Fixes #8

Note:
Status checks for branch policy need to be updated after merging this PR since job names have changed -> will take care about it.
Actions previously added to the allowed actions in the branch policy need to be removed again -> will take care about it.

Type of change

  • 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)
  • This change requires a documentation update

How Has This Been Tested?

PR
https://app.codecov.io/gh/ZEISS/czicompress/tree/8-use-codecov-to-monitor-code-coverage
There is no component-based badge available yet, i.e. average coverage for czishrink and czicompress are displayed and badge. However, when clicking on badge, codecov dashboard shows everything.

Checklist:

  • I followed the Contributing Guidelines.
  • I did a self-review.
  • I commented my code, particularly in hard-to-understand areas.
  • I updated the documentation.
  • I updated the version of czicompress following the README (Versioning) depending on the type of change
    • Bug fix -> PATCH
    • New feature -> MINOR
    • Breaking change -> MAJOR
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.

@FelixS90 FelixS90 linked an issue Nov 9, 2023 that may be closed by this pull request
Copy link

codecov bot commented Nov 9, 2023

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

.github/workflows/czicompress_cmake.yml Outdated Show resolved Hide resolved
.github/workflows/czicompress_cmake.yml Outdated Show resolved Hide resolved
.github/workflows/czishrink_dotnet.yml Outdated Show resolved Hide resolved
@m-ringler m-ringler added the cla Contributor License Agreement sent to Admin label Nov 10, 2023
@FelixS90 FelixS90 force-pushed the 8-use-codecov-to-monitor-code-coverage branch from 0cf170a to 88b004d Compare November 10, 2023 15:36
@FelixS90 FelixS90 marked this pull request as ready for review November 10, 2023 15:39
m-ringler
m-ringler previously approved these changes Nov 10, 2023
czicompress/README.md Outdated Show resolved Hide resolved
@FelixS90 FelixS90 merged commit 362ee5f into main Nov 13, 2023
19 checks passed
@FelixS90 FelixS90 deleted the 8-use-codecov-to-monitor-code-coverage branch November 13, 2023 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla Contributor License Agreement sent to Admin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use codecov to monitor code coverage
2 participants