Yes, I'll start to use it no problem, but what about new contributors on near future?. Like I said, which are gonna be the guidelines for them, or the validation rules, this is resolved using just jshint to validate basic criteria and that's it?
Good point, the code that's not JS. At the wiki page, it's just covered guidelines for JS, not for any other languages. 2014-04-24 16:22 GMT-05:00 Jesse <[email protected]>: > I agree that jshint is awesome and powerful, I use it all the time, and so > should you. > That seems like a great end to this discussion ... > Otherwise, how do we address the 50% of our code that is not JS? > > @purplecabbage > risingj.com > > > On Thu, Apr 24, 2014 at 2:15 PM, Martin Gonzalez Glez < > [email protected]> wrote: > > > jshint is really cool. Agree, +1 for jshint. > > > > A couple of questions about this, which are gonna be the validation > rules? > > I mean, we have to be in the same page about which are gonna be those > > rules. > > Besides, how the users are going to use it? First they need to have the > > node module installed, and then, use a set of validation rules that we > are > > going to provide? > > > > How it's gonna be? use a .jshintrc file with all the rules, general rules > > provided by the community, or each cordova component would contain a > > jshintConfig/package.json with specific validation rules. CLI+rules, > > platforms + rules, plugins + rules > > It's gonna be optional(recommended)or as requirement to deliver code, the > > usage of jshint before commit files? the contributors would need send any > > evidence, that proves that their code has passed the validation? > > Any thoughts? > > > > Jshint, it's awesome and a really good solution, I really like it. > > > > -Martin. > > > > > > 2014-04-24 10:55 GMT-05:00 Marcel Kinard <[email protected]>: > > > > > Big +1 for jshint. > > > > > > On Apr 23, 2014, at 5:53 PM, Brian LeRoux <[email protected]> wrote: > > > > > > > This may get rotten eggs thrown at me but I think it would be good to > > add > > > > jshint to the project. > > > > > >
