jaegertracing / jaeger-client-node / 1135

97%

push

travis-ci

Prettify .js, .json files and add commit hook for auto formatting (#200)

* Prettify .js, .json  files

Pin Prettier v1.10.2 as dev dependency
Add format script to package.json
Close #198
Signed-off-by: Kara de la Marck <karadelamarck@gmail.com>

* Configure so Prettier controls format linting

Update eslint devDependencies
Signed-off-by: Kara de la Marck <karadelamarck@gmail.com>

* Edit flow config to ignore eslint-plugin-jsx-a11y

Signed-off-by: Kara de la Marck <karadelamarck@gmail.com>

* Add devDependency eslint-plugin-jsx-a11y

Signed-off-by: Kara de la Marck <karadelamarck@gmail.com>

* Lint on LTS node versions

Signed-off-by: Kara de la Marck <karadelamarck@gmail.com>

* Add pre-commit hook for formatting

Add devDependencies husky and lint-staged
Signed-off-by: Kara de la Marck <karadelamarck@gmail.com>

* Remove redundant before_script re linting on LTS node

Linting on node 4 alreaady ensured by travis.yml
Signed-off-by: Kara de la Marck <karadelamarck@gmail.com>

* Ignore husky in before_script reinstallation of modules

Signed-off-by: Kara de la Marck <karadelamarck@gmail.com>

* Edit ignore husky command travis.yml

Signed-off-by: Kara de la Marck <karadelamarck@gmail.com>

* Edit before_script back to previous npm install

Signed-off-by: Kara de la Marck <karadelamarck@gmail.com>

* Move husky to optionalDependency

Final npm install in before_script run with --no-optional flag
Signed-off-by: Kara de la Marck <karadelamarck@gmail.com>

* Lint on LTS node versions

Signed-off-by: Kara de la Marck <karadelamarck@gmail.com>

* Remove commented out run lint

Removed from test-without-build in Makefile
We are now linting on LTS versions of Node, see travis.yml
Signed-off-by: Kara de la Marck <karadelamarck@gmail.com>

516 of 560 branches covered (92.14%)

Branch coverage included in aggregate %.

972 of 972 new or added lines in 32 files covered. (100.0%)

1136 of 1145 relevant lines covered (99.21%)

48.1 hits per line