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

Ant Task always displays <option>=false for options loaded via optionsFile attribute #24

Open
stevematyas opened this issue May 4, 2013 · 0 comments

Comments

@stevematyas
Copy link

Using

While the jshint options (checks) are set to true so the errors are reported properly though the console output is misleading shows =false for ALL custom options ?!?!

Context: Using ede60c0 (3.6-SNAPSHOT)

   [jshint] Using custom options:forin=false,plusplus=false,white=false,globalstrict=false,validthis=false,smarttabs=false,browser=false,passfail=false,onevar=false,laxbreak=false,asi=false,maxstatements=false,es5=false,curly=false,rhino=false,proto=false,worker=false,supernew=false,scripturl=false,indent=false,devel=false,maxlen=false,immed=false,maxparams=false,evil=false,iterator=false,camelcase=false,latedef=false,eqnull=false,eqeqeq=false,dojo=false,sub=false,prototypejs=false,debug=false,nonew=false,couch=false,undef=true,multistr=false,noempty=false,wsh=false,mootools=false,node=false,jquery=false,boss=false,shadow=false,funcscope=false,newcap=false,yui=false,strict=false,nomen=false,esnext=false,nonstandard=false,loopfunc=false,bitwise=false,lastsemic=false,maxcomplexity=false,expr=false,quotmark=false,maxerr=false,trailing=false,laxcomma=false,maxdepth=false,noarg=false,unused=false
   [jshint] Using custom options:
   [jshint] 23  JS files passed JSHint code checks

Below is an example jshint v1.1.0 rc file (must strip comments and "predef" : [ ] from https://github.com/jshint/jshint/blob/1.1.0/examples/.jshintrc otherwise ant-jshint is unable to load file -- Jackson deserialization can't handle JS comments or [])

{

    "maxerr"        : 50,       

    "bitwise"       : true,     
    "camelcase"     : false,    
    "curly"         : true,     
    "eqeqeq"        : true,     
    "forin"         : true,     
    "immed"         : false,    
    "indent"        : 4,        
    "latedef"       : false,    
    "newcap"        : false,    
    "noarg"         : true,     
    "noempty"       : true,     
    "nonew"         : false,    
    "plusplus"      : false,    
    "quotmark"      : false,    




    "undef"         : true,     
    "unused"        : true,     
    "unused"        : true,     
    "strict"        : true,     
    "trailing"      : false,    
    "maxparams"     : false,    
    "maxdepth"      : false,    
    "maxstatements" : false,    
    "maxcomplexity" : false,    
    "maxlen"        : false,    

    "asi"           : false,     
    "boss"          : false,     
    "debug"         : false,     
    "eqnull"        : false,     
    "es5"           : false,     
    "esnext"        : false,     
    "evil"          : true,      
    "expr"          : false,     
    "funcscope"     : false,     
    "globalstrict"  : false,     
    "iterator"      : false,     
    "lastsemic"     : false,     
    "laxbreak"      : false,     
    "laxcomma"      : false,     
    "loopfunc"      : false,     
    "multistr"      : false,     
    "proto"         : false,     
    "scripturl"     : false,     
    "smarttabs"     : false,     
    "shadow"        : false,     
    "sub"           : false,     
    "supernew"      : false,     
    "validthis"     : false,     

    "browser"       : true,     
    "couch"         : false,    
    "devel"         : false,     
    "dojo"          : false,    
    "jquery"        : false,    
    "mootools"      : false,    
    "node"          : false,    
    "nonstandard"   : false,    
    "prototypejs"   : false,    
    "rhino"         : false,    
    "worker"        : false,    
    "wsh"           : false,    
    "yui"           : false,    

    "nomen"         : false,    
    "onevar"        : false,    
    "passfail"      : false,    
    "white"         : false 


}
codecop added a commit to codecop/ant-jshint that referenced this issue Jun 29, 2013
options to Map<Object, Object> instead of Properties because wrong
methods were called on Properties. The properties is only used for the
load method, so outside it can be a Map. This solves Issue philmander#19 and Issue
philmander#24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant