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

Add code coverage to blueprint-operator BOP-1133 #100

Merged
merged 4 commits into from
Dec 11, 2024

Conversation

tppolkow
Copy link
Contributor

@tppolkow tppolkow commented Dec 10, 2024

Essentially the same as other PRs adding code-coverage, but some minor changes to accomodate blueprint-operator such as changing root-package to github.com/mirantiscontainers/blueprint-operator/. Also changed the baseline-workflow ref to test.yml as it is not unit.yml like in all our other repos.

The code coverage action is expected to fail on this PR since there is no coverage on main to compare to. Will do a full validation of the action once PR is merged.

@tppolkow tppolkow requested a review from a team December 10, 2024 22:01
Copy link
Contributor

@sakshisharma84 sakshisharma84 left a comment

Choose a reason for hiding this comment

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

looks good!

@tppolkow tppolkow merged commit 5e91c51 into Mirantis:main Dec 11, 2024
5 of 6 checks passed
@tppolkow tppolkow deleted the BOP-1133-add-code-coverage-reporting branch December 11, 2024 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants