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

Fix codecov to ignore 3rdparty folder #26

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

henrygerardmoore
Copy link
Collaborator

@henrygerardmoore henrygerardmoore commented Sep 13, 2024

It looks like we're trying to ignore codecov for the 3rd party folder, but it's included nonetheless. This PR fixes that to get a more accurate codecov value

Currently:
image

After this PR:
[trying to get codecov to cooperate]

@henrygerardmoore henrygerardmoore force-pushed the codecov_ignore_3rdparty branch 14 times, most recently from 72bfc92 to 4a25d66 Compare September 13, 2024 22:43
@facontidavide
Copy link
Collaborator

Looks good to me, this is something I was struggling with!
Let me know when it is ready for review

@henrygerardmoore
Copy link
Collaborator Author

Looks good to me, this is something I was struggling with! Let me know when it is ready for review

I tried a million different things and couldn't get it 🫠 ! I will let you know when it's ready though 👍

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