You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.
I got a flow errors that splits up in 3 messages (blame + comment + blame), this is a single error. but the way linter-flow handles it is it displays an error for both blames, which give exactly the same output
[warning] identifier `EventEmitter` Expected polymorphic type instead of module `events`at line 20 col 24
@MakuraYami This behavior has been included on purpose. It may be a little more noisy, but it makes it easier to see errors as they happen.
Specially in React, if you fail to pass some required props, the error originates at the Component definition. This way, there is also an error where you use the component where the real error usually lies.
Ah, I did not know it would give two different contexts on the same error. I have not tried flow with react all that much, it seemed logical since flow has a array of errors that each one represents one line. Doesn't my separating context's still solve that? if it gives two different context, it will still be 2 Linter messages. There is just no point displaying 2 exactly the same messages when its not these edge cases. If not at the location I showed then later in the process before passing them off to atom, filter out so the messages are unique.
I'll look into it. There is no perfect solution here. Even in non-React code. You can often call a function with the wrong arguments and the error can originate at the function definition. If the function is defined in a different module, you won't even know you called the function wrong till much later.
So, showing the multiple errors still helps. It can get noisy though, and maybe we can use some heuristic to make it less noisy.
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
I got a flow errors that splits up in 3 messages (blame + comment + blame), this is a single error. but the way linter-flow handles it is it displays an error for both blames, which give exactly the same output
here is my error for reference
I tried myself and changed messages.js:26
to
and that solves the issue.
The text was updated successfully, but these errors were encountered: