cjdev2 / jshint-mojo

A simple, speedy maven plugin for running javascript code through jshint
GNU General Public License v2.0
40 stars 32 forks source link

configFile with absolute path #43

Open Sebl29 opened 10 years ago

Sebl29 commented 10 years ago

Hi, in a big project with many projects that include JS-files I want to use one config file for all of them (more specific: every project which has files in a directory described in "directories"). So I would like to define a path in my parent pom.xml to one configuration file that would be used in every project. Currently the configuration value of "configFile" only gets interpreted relative to the basedir, ignoring the leading "/". Tested in 1.3.0 and 1.3.1-SNAPSHOT. Thanks for reading, good project!

Sebl29 commented 10 years ago

I fixed that one in a fork, test included. https://github.com/Sebl29/jshint-mojo

Another fix: findJshintrc (searches in all parent folders for .jshintrc) only gets called if no other file is specified. So it's faster and does not take such an alternative file when it is not desired.