there is no code coverage

[iancrowther]

we need to complete the tests to achieve 100% code coverage

[mike182uk]

There have been some tests contributed. We can generate code coverage using istanbul but where should this coverage be visible? I would not recommend the coverage being in the repo itself. I would suggest using something like codeclimate (which already looks to be used by gitevents/core. This would also probably require a build step (using travis ci or something similar) to have the code coverage sent to codeclimate on new changes.

[iancrowther]

"I would not recommend the coverage being in the repo itself" - how come? surely it just an output in the test report

[iancrowther]

whats wrong with https://www.npmjs.com/package/istanbul?

[iancrowther]

codeclimate for me is candy (which im fine to add) however we should be able to get these metrics without the dependancy of saas tools

[PatrickHeneise]

👍 istanbul should be configured with the project; codeclimate is nice to have (free for open source)

[mike182uk]

@iancrowther In the issue i suggested Istanbul...

I meant i would not store the output from the tool in the repo itself. Why? Because i don't really see a need for this output to be in the repo itself. It just bloats the repo and easily become out of date if its not generated for every single change. Why not just send the coverage to codeclimate and let that keep track of it for us?