fdecampredon / brackets-epic-linter Goto Github PK

Hi, Can you add license, please?

/* jslint ignore:start /
function(){
var bad_code = null;
}
/ jslint ignore:end */

Ignore comments doing nothing

Hi! It would be really helpful if plugin will have an ability to change linting preferences. Ideally it should be some GUI to do it without changing config files.

I looked at this extension and found that there may be some issues running the extension with the SplitView branch.

A more detailed explanation of what breaks and why it breaks is in the Brackets
Extension Splitview Migration Guide. https://github.com/adobe/brackets/wiki/SplitView-Extension-Migration-Guide#extensions-that-will-break

it looks like you're computing the width for the tip based on the width of #editor-holder which isn't correct anymore. Your extension still works but is probably not functioning correctly.

you can test with the branch "jeff/splitview-factory". Let us know if you need more info.

Upcoming Lodash 3.x Update

Dear Brackets extension developer,
we're planning to update the version of Lodash used in Brackets to 3.x (currently, we use 2.4.1).
As we've had a look at your extension, we noticed there's some code that may cause issues while migrating. Thus, we suggest to update your extension so the migration can happen frictionless.
For your extension, we've identified this issue:
Multiple Places: You use _.flatten, which won't default to deep flattening any more. Please see whether you can continue _.flatten or switch over to _.flatten(array, !!_.flattenDeep) for migration

Further information:
Lodash compatibility warnings
Brackets issue about updating Lodash

I've been trying out this extension and noticed that it conflicts with .jshintrc files used by the JSHint plugin here: https://github.com/cfjedimaster/brackets-jshint

I'm guessing this somehow occurs because Epic Linter has its own .jshintrc that I assume overrides any others. A specific example of this being problematic is when one wishes to see warnings when console is used, because the Epic Linter .jshintrc always sets it as a permitted global

Perhaps it'd be good to look into using JSHint's design of recursive overriding, i.e. a .jshintrc in a directory will affect anything below that directory but .jshintrc files in child directories can override, ad infinitum.

Concretely, I would expect a .jshintrc file anywhere in my project structure to work as intended.

How can I turn off the error marks in the scroll bar area?

Alos, they don't update when the scroll area moves.

If I fix an error, the linter still shows it as an error:

In this case, the all caps color definition is now all caps.

jumping to errors when scrollPastEnd is enabled and the error is near the bottom of the page results in jumping below where the actual error is.