-
Notifications
You must be signed in to change notification settings - Fork 30.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Linter does not find all errors that the browsers do #3140
Comments
Given that there are now high quality linters for JS available (eslint, jshint) and there are also extensions for these linters with Code available, the will deprecate the built-in linter support in the next update. Is this decision a problem for you? Can you try the eslint or jshint code extensions? |
What about Type Script? (not using it yet in this project but will) |
Based on the download# of the extensions I'd say eslint, it has 25% more downloads.
Actually the JavaScript language support is powered by TS. TypeScript 1.8 will add more linting checks. If you want linting for TypeScript then we recommend to use tslint (for which there is also a Code extension. |
Well i think you will kill code if you remove the linting. On Fri, Feb 19, 2016 at 11:44 AM, Erich Gamma [email protected]
Justin Romaine |
You need this to be instantly useful for editing JS in my opinion :) On Fri, Feb 19, 2016 at 12:07 PM, Justin Romaine [email protected]
Justin Romaine |
Thanks for your open feedback. I agree that the out of the box experience isn't that nice anymore. I'll try to some fixing/band-aid in the change notes docs for now. We can help with the setup of eslint by providing you with an action to create an eslintrc. At the same the Salsa team has started discussion on providing more warnings. Pls see: microsoft/TypeScript#6802. |
Another option is that we bundle vscode-eslint |
a
continue;
that is not within afor
should be an error.The text was updated successfully, but these errors were encountered: