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 codecov.yml to ensure adequate patch coverage #116

Merged
merged 1 commit into from
Nov 13, 2023
Merged

Add codecov.yml to ensure adequate patch coverage #116

merged 1 commit into from
Nov 13, 2023

Conversation

thunderboltsid
Copy link
Contributor

@thunderboltsid thunderboltsid commented Nov 9, 2023

This sets the minimum patch coverage required for changes to repo to 75%.
Existing code coverage is currently ~77% which is slightly above 75%.

There was a discrepancy between the Go version in go.mod
and the go version in github actions workflow. This change also
alleviates that discrepancy by changing the Go version in both to
the latest Go version (v1.21).

How has this been tested?
Added a dummy change to this PR that drops the overall coverage and has no patch coverage. this caused two failures (overall coverage drop & patch coverage below threshold).

Screenshot 2023-11-10 at 17 10 10 Screenshot 2023-11-10 at 17 06 22

Then removed the offending piece of dummy change and observed the codecov checks pass.

Screenshot 2023-11-10 at 17 52 56

@codecov-commenter
Copy link

codecov-commenter commented Nov 9, 2023

Codecov Report

Merging #116 (eb1b7ee) into main (9fc3562) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #116   +/-   ##
=======================================
  Coverage   77.87%   77.87%           
=======================================
  Files           6        6           
  Lines         330      330           
=======================================
  Hits          257      257           
  Misses         73       73           

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@thunderboltsid thunderboltsid changed the title Add codecov.yml to ensure adequate patch coverage [WIP] Add codecov.yml to ensure adequate patch coverage Nov 9, 2023
@thunderboltsid thunderboltsid changed the title [WIP] Add codecov.yml to ensure adequate patch coverage Add codecov.yml to ensure adequate patch coverage Nov 10, 2023
@thunderboltsid
Copy link
Contributor Author

build-container job is failing due to a Trivy scan stemming from CVE-2023-45142. Attempting to fix that in another PR #117

Copy link
Contributor

@tuxtof tuxtof left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@adiantum adiantum left a comment

Choose a reason for hiding this comment

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

Please add explanation to the commit comment why are go.mod & go.sum changed.

This sets the minimum patch coverage required for changes to repo
to 75%. There was a discrepancy between the Go version in go.mod
and the go version in github actions workflow. This change also
alleviates that discrepancy by changing the Go version in both to
the latest Go version (v1.21).
Copy link

✅ None of your dependencies violate policy!

Copy link

@adiantum adiantum left a comment

Choose a reason for hiding this comment

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

LGTM

@tuxtof tuxtof merged commit 686c526 into main Nov 13, 2023
9 of 10 checks passed
@tuxtof tuxtof deleted the KRBN-7734 branch November 13, 2023 14:40
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.

None yet

4 participants