search

73rhodes / Sublime-JSLint

JSLint for Sublime Text 2 and 3.
80 stars 14 forks source link

Expected indentation suddenly changed from 2 to 4 space #48

Open truh opened 9 years ago

truh commented 9 years ago

I now have hundreds of errors in all of my files which were yesterday passing with hardly any warnings.

YorkshireKev commented 9 years ago

I also have this error with the latest release of jslint. The indentation appears to be fixed at 4, where I prefer to use 2 spaces.

I tried restoring the indent option in JSLint.sublime-settings but this didn't work. I can't see what else has changed that would break this. Maybe jslint.js has changed?

truh commented 9 years ago

JSLint version has changed according to #49. I described a workaround in #47.

rangercyh commented 9 years ago

what a pity!!!! I am unhappy now!!

liberborn commented 9 years ago

It's not only about indentation. I use 4 spaces indentation. But so many errors are logged now I could even know how to report them to you.

Fasani commented 9 years ago

I am happy to use the new version but how to change my settings file so that it start to work again.

73rhodes commented 9 years ago

Please log issues with JSLint and ask Doug Crockford to stop introducing build-breaking non-backwards compatible changes.

truh commented 9 years ago

SublimeLinter-jslint uses node-jslint and doesn't have these issues. If I had installed Crockfords software I would have issued the bug report to him. But since I installed Sublime-JSLint and not jslint.js this is in my opinion the logical place to report to.

73rhodes commented 9 years ago

Sorry, but you did install jslint.js - it's included in this package. Sublime-JSLint is simply a wrapper around JSLint and the objective is to keep it reasonably up-to-date with the latest version of JSLint. You're free to use whatever version of JSLint you want, though - it's easy to configure the plugin to point to another version of jslint elsewhere on your system and not necessarily the built-in one. FWIW I agree that forcing a 4-space indent on everyone was not a good idea.

73rhodes commented 9 years ago

Refer to https://github.com/douglascrockford/JSLint/commit/f819cf3c65d07086cf217c549c577fa77dc65fc9#commitcomment-14603692