add & configure jshint for build task

graememcc / micropolisJS

A port of Micropolis to JS/HTML5

Other

639 stars 94 forks source link

add & configure jshint for build task #22

Closed ian-moore closed 9 years ago

ian-moore commented 9 years ago

Fixes #19

I added grunt-contrib-jshint to run on the build task. After looking at the results, I turned off newcap option because it popped up all over the place and doesn't look like you're using that convention. The only other thing it found was a switch case that falls through that it needed to ignore.

I also added load-grunt-tasks to autoload the grunt tasks.

graememcc commented 9 years ago

Yeah, I prefer new agnostic constructors. Didn't know about load-grunt-tasks, so thanks for that tip!

ian-moore commented 9 years ago

No problem! Are you looking for help on any of the other open issues?