ChainSafe / discv5

A Typescript implementation of the Discv5 protocol
Apache License 2.0
28 stars 15 forks source link

Add c8 based coverage reports for e2e and unit tests #244

Open 0xTylerHolmes opened 1 year ago

0xTylerHolmes commented 1 year ago

This PR adds c8 based coverage reports

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

dapplion commented 1 year ago

Should the coverage report be acted upon? i.e. upload to some remote tracker

0xTylerHolmes commented 1 year ago

I left it as is just to gain some quick metrics and keep it simple.

If you want I could look into a good way to automate this and make another PR.

dapplion commented 1 year ago

I left it as is just to gain some quick metrics and keep it simple.

If you want I could look into a good way to automate this and make another PR.

CC @wemeetagain

wemeetagain commented 2 weeks ago

looks good, we just need to update the c8 version (now at 10.x)