search

boostorg / multiprecision

Boost.Multiprecision
Boost Software License 1.0
195 stars 112 forks source link

Add codecov run to CI #581

Closed mborland closed 9 months ago

codecov[bot] commented 9 months ago

Welcome to Codecov :tada:

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 :open_umbrella:

mborland commented 9 months ago

@ckormanyos Anything you would add or remove from this?

ckormanyos commented 9 months ago

Anything you would add or remove from this?

Hi Matt (@mborland) I think this is great, both here as well as in math. And it is really cool to see those coverage numbers above $90\%%$.

You will chuckle, but I'd kind of like to show the coverage with our codecov-badge. I know some folks do not really like third party badges so much. But I think they add value as we strive toward the upper nineties, ... unless there is some poicy against them?

Thanks for digging into this Matt´!

ckormanyos commented 9 months ago

This is the badge we got so far. It is great to be above $90\%$!...

codecov

ckormanyos commented 9 months ago

If you do decide to show the badges, then where? I don't really know. Maybe a third row in the table on the front page Markdown? But we won't have any data from master for a while...

mborland commented 9 months ago

If you do decide to show the badges, then where? I don't really know. Maybe a third row in the table on the front page Markdown? But we won't have any data from master for a while...

I think a third row on the table is the way to go. I'll hold off adding the badges until we have data from master.