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

Add code coverage to CI (+ a badge) #330

Open
tasn opened this issue Mar 21, 2022 · 4 comments · May be fixed by #448
Open

Add code coverage to CI (+ a badge) #330

tasn opened this issue Mar 21, 2022 · 4 comments · May be fixed by #448
Labels
server Issues regarding the server component

Comments

@tasn
Copy link
Member

tasn commented Mar 21, 2022

Multiple options, not sure what's the right one:

@tasn tasn added the server Issues regarding the server component label Mar 21, 2022
@tasn tasn linked a pull request May 23, 2022 that will close this issue
@michaelgrigoryan25
Copy link

What about using a GitHub Action? I think I found something at https://github.com/marketplace/actions/rust-grcov which uses grcov under the hood.

@tasn
Copy link
Member Author

tasn commented Jun 20, 2022

Thanks for the feedback! Yeah, it looks promising. We also want to have something that works locally, and we want to merge it with the test suite runs, but I'm sure we can get it working.

@michaelgrigoryan25
Copy link

In that case it may be possible to come up with something custom. cargo has built-in support for extending its functionality.

@tasn
Copy link
Member Author

tasn commented Jun 20, 2022

Yeah. I just hope tarpaulin will prove itself as the easy solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
server Issues regarding the server component
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants