Skip to content
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

tslint not working and how remove warning on disabled autocompile & #126

Closed
Master-01 opened this issue Dec 22, 2016 · 2 comments
Closed

Comments

@Master-01
Copy link

Hellow!

  1. I enabled Preferences -> JavaScript -> TypeScript -> Validation -> Search for 'tslint' from folder (or manualy put path to file tslint.json)
    I copy to Project folder file tslint.json

body tslint.json
{
"rules": {
"class-name": true,
"semicolon": [true, "always"]
}
}

but TypeScript ignoring line without semicolon like this var a="test"
How to include a warning that there is no bar in the semicolon?

  1. How to disable a warning on the first line "TypeScript file cannot be compiled on save because tsconfig.json disable it. If this is not intended, please set "compileOnSave" to "true" of your "tsconfig.json" file." ?
    I specifically do not include the compilation, and do not want to see the warning about it!
@angelozerr angelozerr changed the title jslint not working and how remove warning on disabled autocompile & tslint not working and how remove warning on disabled autocompile & Dec 22, 2016
@angelozerr
Copy link
Owner

but TypeScript ignoring line without semicolon like this var a="test"

Integration of tslint inside typescipt.java is not very good:

  • it works only when you activate TypeScript builder.
  • it works only when you save the editor (as you type validation for tslint doesn't work)
  • it is very slow because each time you save the editor, it executes tslint process: tslint config is loaded, file which is validated is recompiled, etc

To improve the tslint integration (as you type validation, have good performance, etc) we need to wait for microsoft/TypeScript#10162 microsoft/TypeScript#6508

And we could even support tslint quick fixes like https://github.com/angelozerr/typescript.java/wiki/New-and-Noteworthy-1.2.0#quick-fixes

I specifically do not include the compilation, and do not want to see the warning about it!

You cannot, please create a new issue with that and please share your project which causes your problem.

@angelozerr
Copy link
Owner

@angelozerr angelozerr added this to the 1.2.0 milestone Feb 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants