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

coverage: add coverage CI action #1343

Merged
merged 1 commit into from
Nov 21, 2023

Conversation

SuperAuguste
Copy link
Member

No description provided.

@SuperAuguste SuperAuguste force-pushed the auguste/code-coverage-notification branch 9 times, most recently from d5fcbe7 to 2828d86 Compare July 21, 2023 18:39
@zigtools zigtools deleted a comment from github-actions bot Jul 21, 2023
@SuperAuguste SuperAuguste force-pushed the auguste/code-coverage-notification branch 12 times, most recently from 52bb259 to 3319ecd Compare July 21, 2023 20:53
@SuperAuguste
Copy link
Member Author

Hmm I'm running into this incredibly weird kcov behavior where it occasionally gives me fantastic results... and sometimes now (see above) - seems like it could be:

  1. kcov flakiness
  2. build.zig issue
  3. GitHub Actions configuration thing?

@SuperAuguste SuperAuguste force-pushed the auguste/code-coverage-notification branch 7 times, most recently from 26258c5 to 0970e4b Compare July 22, 2023 15:28
@SuperAuguste SuperAuguste force-pushed the auguste/code-coverage-notification branch 3 times, most recently from 1860d84 to eea32b4 Compare July 22, 2023 17:39
@zigtools zigtools deleted a comment from github-actions bot Jul 22, 2023
@SuperAuguste SuperAuguste force-pushed the auguste/code-coverage-notification branch from eea32b4 to b852779 Compare November 21, 2023 07:04
@zigtools zigtools deleted a comment from github-actions bot Nov 21, 2023
@SuperAuguste SuperAuguste force-pushed the auguste/code-coverage-notification branch 7 times, most recently from 469f472 to a24bee9 Compare November 21, 2023 07:37
Copy link

codecov bot commented Nov 21, 2023

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

@SuperAuguste SuperAuguste force-pushed the auguste/code-coverage-notification branch 4 times, most recently from bd14247 to 960c4e5 Compare November 21, 2023 08:21
@SuperAuguste SuperAuguste marked this pull request as ready for review November 21, 2023 08:24
@SuperAuguste SuperAuguste force-pushed the auguste/code-coverage-notification branch from 960c4e5 to 8b285d3 Compare November 21, 2023 08:27
@SuperAuguste
Copy link
Member Author

Wow this was quite the adventure, hopefully it'll be helpful :)

@SuperAuguste SuperAuguste enabled auto-merge (squash) November 21, 2023 08:29
@SuperAuguste SuperAuguste force-pushed the auguste/code-coverage-notification branch from 8b285d3 to 5e90bc6 Compare November 21, 2023 17:29
@SuperAuguste SuperAuguste force-pushed the auguste/code-coverage-notification branch from 5e90bc6 to a5c4876 Compare November 21, 2023 17:34
@SuperAuguste
Copy link
Member Author

this seems fine, gonna yolo merge it

@SuperAuguste SuperAuguste merged commit 8d162d3 into master Nov 21, 2023
4 checks passed
@SuperAuguste SuperAuguste deleted the auguste/code-coverage-notification branch November 21, 2023 17:36
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