-
Notifications
You must be signed in to change notification settings - Fork 26
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 work #159
Comments
conflicts with brackets-git |
@spietrek Sorry about that. Have you tried switching the order your linter definition to:
Interactive linter only supports one linter, even though the definition is an array. Array definition is just thinking ahead of features I would like to add and have not had to time to. @iegik I am interested in what you said here. Is there a known conflict, something you have ran into, or just a guess? |
Thanks Miguel. I am also running Brackets Git too. -Steve Pietrek
|
@spietrek Did you try switching the order of your linter definitions to have jshint first? |
@MiguelCastillo I have tried to switch the order of extensions. No effect. |
The actual bit that makes assumptions about there being an array and using the first entry is https://github.com/MiguelCastillo/Brackets-InteractiveLinter/blob/master/linterManager.js#L36 This issue certainly does not seem straight forward, so if you guys could help me diagnose it, that would be awesome. What environments (OS) are you guys running? The other thing that is interesting is
That's not needed unless you have install the corresponding extensions and not using interactive linter. Interactive linter will override the JS linting in Brackets. |
Ubuntu. I have installed both extensions -
|
Ohhh, that's interesting (in a bad way). Interactive linter works when brackets-git is not installed? :-/ I gotta look into that soon! Thanks for the info. |
I have seen Interactive Linter not work on OSX. I also have Brackets Git installed. Seems to happen more often after installing or uninstalling other extensions and restarting Brackets. Sometimes takes multiple restarts with and without extensions to get things to work. -Steve Pietrek
|
@MiguelCastillo, @spietrek I have removed all extensions except Git and Linter and switching them on/off, decided, that Linter does not work after Git is enabled. |
This is really helpful guys. I really don't run into these issues. I will try to repro with git. Thanks!! |
Note: Git hightlight row corner then line was changed, but Linter also mark that line, when it is with the error. I think marks are shown there, but they are not visible because Git replace/override them! |
I have found that if you disable the extension and then enable it again the linter starts to work again. This is with brackets-git also installed. Thanks!! |
I am having an issue where the linter extension does not work. I have JSCS and JSHint loaded. The linter will work only occasionally. Generally, I need to restart Brackets a bunch of times with and without extensions in order for it to activate.
"language": {
"javascript": {
"linting.prefer": [
"JSHint",
"JSCS"
],
"linting.usePreferredOnly": true
}
},
"interactive-linter.javascript": [
"jscs",
"jshint"
],
"interactive-linter.delay": 1000,
"interactive-linter.html": [
"htmlhint"
],
The other extensions I have loaded include:
What should I look out for to determine why the extension is not working? Anything I should be aware of?
Thanks
-Steve
The text was updated successfully, but these errors were encountered: