-
-
Notifications
You must be signed in to change notification settings - Fork 153
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
use-flow-type marks a reference as used with the wrong reference when names are the same #340
Comments
@mruzekw Can you give a try fixing this? |
Sure. I don't know what the full scope of the bug is. Is it all |
Couldn't tell without digging into it myself. |
So it looks like this only happens when a module def is the same name as a function param. Here are a couple of examples where this takes affect:
|
Alright, in an attempt to fix this I've found that Not sure how to proceed here other than raise an issue in the ESLint repo. |
I may be missing something, but would it help if this scope lookup started one scope up? |
@pnevyk That's how I attempted to fix it. But notice the https://github.com/eslint/eslint/blob/master/lib/linter.js#L660-L678 |
@mruzekw Thanks for clarification. |
Code
Versions
Expected
No errors.
Actual
The text was updated successfully, but these errors were encountered: