jshint / fixmyjs

Automatically fix silly lint errors.
http://fixmyjs.com
Other
971 stars 78 forks source link

indent size does not work in legacy mode #121

Closed ruanyl closed 9 years ago

ruanyl commented 9 years ago

Is it a known issue?

goatslacker commented 9 years ago

can you post an example? It's working fine on fixmyjs.com

ruanyl commented 9 years ago

@goatslacker did you run it under legacy mode? It does not work for me even on fixmyjs.com when i checked "Use Legacy". Did i missed something?

i checked "Use Legacy" and set options indent = 2

goatslacker commented 9 years ago

Ah my bad no I hadn't checked legacy. It doesn't work because jshint removed whitespace checking a while ago.

See: https://github.com/jshint/jshint/commit/12147ed96c66faa638835a6b747b5e9127cb18b0