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

[ci] remove tarpaulin #822

Merged
merged 1 commit into from
Nov 3, 2021
Merged

[ci] remove tarpaulin #822

merged 1 commit into from
Nov 3, 2021

Conversation

woshilapin
Copy link
Contributor

It seems we have a problem with how cargo tarpaulin is called (see here).

image

ArnaudOggy
ArnaudOggy previously approved these changes Oct 27, 2021
patochectp
patochectp previously approved these changes Oct 27, 2021
@woshilapin woshilapin dismissed stale reviews from patochectp and ArnaudOggy via dd8b853 October 27, 2021 12:40
@woshilapin woshilapin force-pushed the fix-tarpaulin branch 11 times, most recently from 1eccdf5 to 437ddd4 Compare October 27, 2021 14:39
@woshilapin
Copy link
Contributor Author

woshilapin commented Oct 27, 2021

I was not able to make this work. I'm guessing it's mostly because of one of the feature (proj or xmllint) but was not able to find a solution. Therefore, I'm proposing to remove this step entirely, since it doesn't work anyway.

@datanel datanel merged commit 00f3d5f into master Nov 3, 2021
@datanel datanel deleted the fix-tarpaulin branch November 3, 2021 09:42
@woshilapin woshilapin changed the title [ci] fix how tarpaulin is called [ci] remove tarpaulin Nov 3, 2021
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.

4 participants