SublimeLinter-flow icon indicating copy to clipboard operation
SublimeLinter-flow copied to clipboard

SublimeLinter-flow doesn't respect a @FlowFixMe comment

Open jdalegonzalez opened this issue 8 years ago • 2 comments

If the following line is in a file, you'll see a flow error warning about userSelect not existing on the style object

var es = (document.documentElement) ? document.documentElement.style.userSelect : null;

The following change should cause the error to go away...

// $FlowFixMe: Flow is wrong.  userSelect exists.  At least in Chrome world.
var es = (document.documentElement) ? document.documentElement.style.userSelect : null;

jdalegonzalez avatar Feb 14 '17 14:02 jdalegonzalez

I believe the issue is that flow does not support error suppressing comments with the check-contents command.

davidaurelio avatar Apr 29 '17 23:04 davidaurelio

Well, that makes it a flow bug, not something particular to this repo right? We could change the behaviour to not use stdin, but what use is supporting stdin in your linter if it's going to ignore config like this?

braver avatar May 17 '18 13:05 braver