Add test coverage reporting#600
Conversation
|
@kkoskelin thanks for putting this together, it's definitely needed. Those numbers are better than I was expecting! @jamhall while I have your attention, could you install codecov to this repo when you have time? I don't think it needs additional setup for collaborator access, and open source licenses are free. Thanks! |
|
@kherock - I have activated codecov. Not too sure what I need to do now? I have a token... I guess you'll need this token once GH actions has been setup? |
fishcharlie
left a comment
There was a problem hiding this comment.
Looks like there are merge conflicts. But other than that, this looks like a great PR!
|
Codecov does have a GitHub action. I don't think that needs to be included in this PR and can be included in a separate PR once we get GitHub actions support up and running. |
|
This is now in the main branch via #681, closing. |
As an aid to ensuring proper coverage, I've installed istanbul and configured coverage numbers that ought to ensure the threshold reflects current coverage and do not slip below those given numbers.
I would like to submit further pull requests with even more test coverage, and this would be a helpful tool.