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.

Sign up for GitHub

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

Jump to bottom

LLVM coverage reporting #132

Merged
merged 1 commit into from
Mar 30, 2023
Merged

LLVM coverage reporting #132

merged 1 commit into from
Mar 30, 2023

Conversation

CrockAgile
Copy link
Collaborator

I saw a repo the other day using LLVM to inject coverage reporting. I thought I would give it a try since grcov having issues.

@CrockAgile CrockAgile self-assigned this Mar 30, 2023
This was referenced Mar 30, 2023
Closed
Closed
@shnewto
Copy link
Owner

shnewto commented Mar 30, 2023

🤩

@CrockAgile CrockAgile marked this pull request as ready for review March 30, 2023 20:13
@CrockAgile CrockAgile requested a review from shnewto March 30, 2023 20:13
@CrockAgile
Copy link
Collaborator Author

maybe a solution for #130 ? idk!

@shnewto
Copy link
Owner

shnewto commented Mar 30, 2023

toootally, I think it's an awesome solution to #130

@CrockAgile
Copy link
Collaborator Author

cool then im gonna go for it. no time like the present 🏃‍♂️

@CrockAgile CrockAgile merged commit 73417e0 into main Mar 30, 2023
@CrockAgile CrockAgile deleted the llvm-coverage branch March 30, 2023 20:24
@shnewto
Copy link
Owner

shnewto commented Mar 30, 2023

🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shnewto shnewto Awaiting requested review from shnewto

Assignees

@CrockAgile CrockAgile

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@CrockAgile @shnewto