I am pro consistency as well, and I prefer including semi-colons shows your intent, where a missing semi-colon could be intentional, or just forgotten.
My ultimate point with this thread was just that I feel the CI tests should be concerned with function, and not form. I honestly don't care if people use or don't use semi-colons, spaces, tabs, trailing spaces, cuddled braces, ... as long as the code is readable, understandable, and demonstrates intent. Anyway, it is all fine, we can move on. @purplecabbage risingj.com On Fri, Aug 22, 2014 at 12:17 PM, Josh Soref <[email protected]> wrote: > Michal Mocny wrote: > >What we can re-evaluate is our .jshintrc defaults: do we want to allow > >missing ; in our codebase? Seems like this is a very divided issue. I'm > >pro semicolons, but I hear thats old-school. I'm also pro-choice ;) so > >I'd > >be happy to remove that option. > > >I'm also pro-consistency so I'd rather we use semicolons everywhere or > >nowhere. > > +1 > >
