search

jaridmargolin / formatter.js

Format html inputs to match a specified pattern
http://firstopinion.github.io/formatter.js
2.48k stars 235 forks source link

Linting fixes. #32

Closed brianpeiris closed 10 years ago

brianpeiris commented 10 years ago

I've fixed some linting issues that were caught by our closure compiler setup. The code used char as a variable in several places. This caused an error since it's considered a reserved word in EcmaScript 3. I also fixed an undeclared variable and a missing semicolon. Lastly, I added some options to the grunt config so that these types of errors can be caught going forward. I also added the concatenated files to the list of linted files since you can't lint the concatenation templates on their own.

jaridmargolin commented 10 years ago

Appreciate the PR! Apologies for the delayed response.

brianpeiris commented 10 years ago

No worries. Thanks for accepting the PR.