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

codecov, fix: Fix the ignoring path regex pattern in codecov.yaml (#175) #221

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

art-tapin
Copy link
Contributor

This PR introduces a fix to path ignoring regexp pattern in codecov.yaml.
The codecov.yaml was invalid because of missing double quotes in "**/*.deepcopy.go" path.

This PR is a part of #175, which just introduces a fix to a path for codecov.yaml so it can work correctly and be applied to the codecov configuration.

Read more at: https://docs.codecov.com/docs/ignoring-paths

@codecov
Copy link

codecov bot commented Aug 1, 2023

Codecov Report

Merging #221 (72138ed) into main (dfad7ee) will increase coverage by 9.47%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #221      +/-   ##
==========================================
+ Coverage   52.40%   61.88%   +9.47%     
==========================================
  Files          34       33       -1     
  Lines        3631     3112     -519     
==========================================
+ Hits         1903     1926      +23     
+ Misses       1532      994     -538     
+ Partials      196      192       -4     
Flag Coverage Δ
integration 66.95% <ø> (+1.43%) ⬆️
unit 56.49% <ø> (+14.46%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
api/v1beta1 (u) 26.86% <ø> (+21.35%) ⬆️
pkg/common (u) 77.27% <ø> (ø)
pkg/istio (u) 29.69% <ø> (ø)
pkg/log (u) 31.81% <ø> (ø)
pkg/reconcilers (u) 40.16% <ø> (ø)
pkg/rlptools (u) 54.47% <ø> (ø)
controllers (i) 66.95% <ø> (+1.43%) ⬆️

see 12 files with indirect coverage changes

Copy link
Contributor

@KevFan KevFan left a comment

Choose a reason for hiding this comment

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

Looks to be working ! 👍
Is the PR ready for review ? 🤔

@art-tapin art-tapin marked this pull request as ready for review August 1, 2023 15:21
@art-tapin art-tapin requested a review from a team as a code owner August 1, 2023 15:21
@art-tapin
Copy link
Contributor Author

Looks to be working ! 👍 Is the PR ready for review ? 🤔

Yes, please! It is now ready! @KevFan

@art-tapin art-tapin requested a review from KevFan August 1, 2023 15:21
Copy link
Contributor

@KevFan KevFan 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 to me ! 🥇

@art-tapin art-tapin merged commit 476e637 into main Aug 1, 2023
@art-tapin art-tapin deleted the codecov_yaml_regexp_fix branch August 1, 2023 15:24
@@ -11,9 +11,10 @@ codecov:
comment:
layout: "header, diff, flags, components, files"

# Find more at https://docs.codecov.com/docs/ignoring-paths
ignore:
- api/external/** # ignoring external vendor code
Copy link
Member

Choose a reason for hiding this comment

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

how is this line not affected?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, DD. I was thinking about providing the direct link to the documentation, but now I am thinking that the link is prone to change. You are right, it was a useless comment 💯

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.

3 participants