search

SublimeLinter / SublimeLinter-flow

SublimeLinter plugin for JavaScript static type checking, using flow.
MIT License
117 stars 24 forks source link

`flow check-contents` causes funny errors #47

Closed LPGhatguy closed 6 years ago

LPGhatguy commented 7 years ago

Flow reports errors differently between flow and flow check-contents in some specific cases! I've started seeing ghost errors reported from SublimeLinter-flow because of it. See facebook/flow#2235 for the upstream issue.

Are there any options available to allow SublimeLinter-flow to work around this? One comment on the linked issue mentions a proposal for better IDE integration. One other comment mentioned that there's an IDE that uses flow for files that have been saved and flow check-contents for unsaved files.

davidaurelio commented 7 years ago

check-contents also does not report “unused suppression” errors.

I hope it can be fixed upstream. Using check and check-contents depending on whether a buffer is saved, and getting different results, might feel very weird.

braver commented 6 years ago

duplicate of #50