search

pgAdmin / eslint-parallel

Tiny eslint wrapper to allow executing javascript linting in parallel.
Apache License 2.0
27 stars 11 forks source link

Passing directories results in all non-js files being globbed #3

Closed itsravenous-sky closed 5 years ago

itsravenous-sky commented 6 years ago

For example, running eslint-parallel ./src will attempt to lint all files under src, whether they be JavaScript or not.

Replacing ./src with ./src/**/*.js will attempt to lint files specified in .eslintignore, causing warnings.

Eslint itself has a utility for constructing the necessary globs, see https://github.com/eslint/eslint/blob/4def8764273e7cc53c71f0dd67da271636e4c1be/lib/util/glob-util.js#L47-L53. It doesn't look like it's exported from the package, but it's a small piece of code to reproduce.

mmmmmmaya commented 6 years ago

Would be interested to hear about updates on this, as it's preventing me from being able to use this otherwise neat library to lint my large projects 😭

alansouzati commented 6 years ago

sure. pull requests are gladly appreciated

alansouzati commented 5 years ago

https://github.com/alansouzati/eslint-parallel/releases/tag/v1.0.0