Closed erikyo closed 1 month ago
hey @johnhooks what do you think? we can merge this one? few lines involved but in that way we can track the new tests coverage
I noticed an issue here... the tests are fired twice because the prepublish
action. we need to change that to prepublishOnly
This PR adds the coverage for tests
long story short, since the most used with mocha is nyc I thought it would work whereas since it is esm you have to use c8
https://github.com/istanbuljs/nyc/issues/1287#issuecomment-706527909