2
100%
master: 100%

Ran 09 Jul 2015 12:53AM UTC

Files 13

Run time 1s

Badge

Embed ▾

Committed 09 Jul 2015 12:53AM UTC coverage: 97.293%. Remained the same

Job # 2006.2

Build Type

push

travis-ci

Committed by Mike Pennisi

Commit Message

Support `--prereq` CLI options

Since [the docs][1] mention it, and it's already supported via [config][2],
I thought I'd just pop the CLI support in.

This PR make `jshint` allow specifying the `--prereq` option on the command
line, accepting a comma-separated list of paths.

Test-related notes:

- Fixes the existing `testPrereq` test to check the right exit code. In its
  current incarnation, it could never fail (even when `lint` didn't pass)
- Added 2 tests for specifying the `--prereq` options from the command line.

Implementation notes:

I did the simplest thing possible (just merging the CLI `prereq` options in
with the config ones in the `run` method in `cli.js`). It's a little bit
awkward because there are 2 paths to `lint`:

1. `opts.useStdin`
2. `!opts.useStdin`

And each loads the config in a slightly different way. So the `mergeCLIPrereq`
function has to be in 2 code paths. So, slightly awkward, but not too terrible.

Fixes #1685.

[1]: http://jshint.com/docs/cli
[2]: https://github.com/jshint/jshint/blob/master/src/cli.js#L451-458

Conflicts:

	src/cli.js
	tests/cli.js

Run Details

3774 of 3879 relevant lines covered (97.29%)

3039.52 hits per line

jshint / jshint / 2006 / 2
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2006.2

jshint / jshint / 2006 / 2 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2006.2

jshint / jshint / 2006 / 2
100%
master: 100%

README BADGES
x