Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Plexus file exclusion patterns #65

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,8 @@ It's real simple and it runs [JSHint](http://www.jshint.com/) on your *.js files
| globals | | List of comma-separated [JSHint globals](http://www.jshint.com/docs/#usage) |
| configFile | | Path to a JSHint JSON config file. Its contents will override values set in `options` and `globals`, if present. Please note that block and line comments will be stripped prior to processing so it's OK to include them. |
| directories | `<directory>src</directory>` | Locations in which the plugin will search for *.js files |
| excludes | | Excludes are resolved relative to the basedir of the module |
| excludes | | Excludes are resolved relative to the source directories. Standard maven semantics apply. |
| includes | "\*\*/\*.js" | Includes are resolved relative to the source directories. Standard maven semantics apply. |
| reporter | | If present, JSHint will generate a reporting file which can be used for some CI tools. Currently, `jslint`, `html`, and `checkstyle` formats are supported. |
| reportFile | target/jshint.xml | Path to an output reporting file |
| failOnError | `true` | Controls whether the plugin fails the build when JSHint is unhappy. Setting this to `false` is discouraged, as it removes most of the benefit of using this plugin. Instead, if you have problem files that you can't fix [disable/override JSHint on a per-file basis](http://www.jshint.com/docs/#config), or tell the plugin to specifically exclude them in the `excludes` section |
Expand Down Expand Up @@ -43,7 +44,11 @@ It's real simple and it runs [JSHint](http://www.jshint.com/) on your *.js files
<excludes>
<exclude>src/main/webapp/hackyScript.js</exclude>
<exclude>src/main/webapp/myDirectoryForThirdyPartyStuff</exclude>
<exclude>**/*.min.js</exclude>
</excludes>
<includes>
<include>**/*.js</include>
</includes>
<reporter>jslint</reporter>
<reportFile>target/jshint.xml</reportFile>
<failOnError>false</failOnError>
Expand Down
Loading