I have read through guide on wiki, as well as jshint config,
but there are few differences between the project's code and guide lines.
Most noticeable are:
- Oneliners a-la
if(true) doSomething
instead ofif(true){doSomething}
. This behavior makes jshint cry, while being presented all around the code. What about just disabling "curly"? - 'strict mode' is presented only on some files. Moreover, there are
var
definitions before scoped function, for example -categories.js
. Is there a reason for it? I would rather think about putting everything in the function's body to be simple and logical way, but mb I'm missing something?