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

Only process and upload code coverage results in two configurations. #152

Merged
merged 14 commits into from
Dec 1, 2024

Conversation

LilithHafner
Copy link
Owner

We should cover everything in one of these two cases.

@LilithHafner
Copy link
Owner Author

We upload somewhere between 10 and 100 builds...

@LilithHafner
Copy link
Owner Author

I sure do wish there was an easier way to find out how many

@LilithHafner
Copy link
Owner Author

15

@LilithHafner
Copy link
Owner Author

Which is the number of matrix entries; looks like the if is broken

@LilithHafner
Copy link
Owner Author

Looks great! Can confirm 2 uploads, and when watching the checks roll in I never saw the codecov checks until both min/x64/ubuntu and 1/x86/ubuntu had finished.

The drop in coverage is likely due to changes in how lines are counted as "lines of code" or not in various versions of Julia. The diff is 3 lines that were previously "covered" are now untracked.

@LilithHafner LilithHafner merged commit 3051d66 into main Dec 1, 2024
22 of 23 checks passed
@LilithHafner LilithHafner deleted the LilithHafner-patch-1 branch December 1, 2024 16:38
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.

1 participant