Coveralls? - Githubissues

syrjs / core

Light Dynamic UI Engine for Native and Web with a React Like API

http://syr.js.org

MIT License

56 stars 24 forks source link

Coveralls? #256

Open brodycj opened 6 years ago

brodycj commented 6 years ago

I noticed that while coveralls is used in Travis CI it would produce no extra output on a passing build (see GH-251 for passing "Node.js" build). I can think of a couple possible solutions:

completely remove coveralls
update coveralls to resolve npm audit issues then configure a reporter or connect to coveralls.io

P.S. ~~What should we do about travis-cov devDep? Can we remove it?~~ (travis-cov is now removed)

dmikey commented 6 years ago

So I think we want to try to solve the conflicts, looking back through the project, it indeed was intended to produce security coverage scans for SonarCloud. I'll put this on a list of things to look at, perhaps it is indeed extraneous. Thanks!