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.

Sign up for GitHub

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

Jump to bottom

Do not display a warning on excluded directories #58

Open
highsource opened this issue Nov 14, 2014 · 2 comments · May be fixed by #65
Open

Do not display a warning on excluded directories #58

highsource opened this issue Nov 14, 2014 · 2 comments · May be fixed by #65

Comments

@highsource
Copy link

When certain directories are excluded in plugin configuration in pom.xml, the plugin outputs warnings:

[INFO] using jshint version 2.4.1
[WARNING] Excluding D:\Projekte\app\trunk\impl\app-war\src\main\webapp\app\js\lib\OpenLayers-2.13.1\lib\deprecated.js
[WARNING] Excluding D:\Projekte\app\trunk\impl\app-war\src\main\webapp\app\js\lib\OpenLayers-2.13.1\lib\Firebug\firebug.js
[WARNING] Excluding D:\Projekte\app\trunk\impl\app-war\src\main\webapp\app\js\lib\OpenLayers-2.13.1\lib\Firebug\firebugx.js

This does not hurt but feels completely unnecessary. I'm excluding a large third-party library and get a couple of hundreds of these messages.
@yas99en
Copy link

yas99en commented Jul 17, 2015

I think so.

@bvarner
Copy link

bvarner commented Oct 30, 2015

These should probably be debug.

bvarner added a commit to e-gineering/jshint-mojo that referenced this issue Oct 30, 2015
@bvarner
Resolve issue cjdev2#58, log exclusion details from ignore files as d…
019c390 
…ebug.
@bvarner bvarner linked a pull request Oct 30, 2015 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use Plexus file exclusion patterns e-gineering/jshint-mojo
3 participants
@bvarner @highsource @yas99en