search

jozefizso / generator-license

Yeoman Generator - License
MIT License
59 stars 20 forks source link

Coveralls gulp #56

Closed ek9 closed 7 years ago

ek9 commented 7 years ago

Add coveralls support. For this I have moved Mocha tests to be executed via Gulp (it sets up Istambul for coverage reporting).

I have also added gulp coveralls after_script in .travis.yml. This will fix #55 and enable coveralls.io integration once owners of the repository set up the link on https://coveralls.io/ website.

This will also allow me to setup easy linting on top of gulp via eslintrc for the next PR to fix #54 .

SBoudrias commented 7 years ago

Hey, I added jest and coverage with https://github.com/jozefizso/generator-license/commit/e2473fb38f1b7e2f364d82d5b56cf51aa83a8c53

As you told you had issue getting it to run, I though I'd go ahead and just make it. I'll let you update the eslint setting to run without extra wrappers :)

ek9 commented 7 years ago

It's up to you, but there are problems with current setup:

  1. It always runs, even on dev machines. Which always results in an error.