-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
The prop value with an expression type of OptionalMemberExpression could not be resolved #70
Comments
I am seeing this with
but if I change it it works. 🤔 Wondering if this is broken for all valid html attrs maybe 🤷♂️ |
Working on it now. I need to update all the dev dependencies to get there :) |
[jsx-eslint#73] Account for SpreadElement AST Nodes [jsx-eslint#70] Support OptionalMemberExpression AST nodes
[#70] Support OptionalMemberExpression AST nodes
@HsuTing , Fixed in version v2.2.1. This will log an error in the console, but it will no longer throw. |
@jessebeach OK, thank for your help. |
Could we not get this error? The message comes from here, following the issue trail from eslint. If there is an error/bug, it should get fixed, if it's a false report, I don't want to see it. |
@MathiasKandelborg I understand the frustration here. I feel it, too. I hate seeing these bugs pop up. I'll look into it this weekend and put out a patch release. |
note that v3.0 of jsx-ast-utils is out, and eslint-plugin-jsx-a11y has been updated with it, but not yet released - so a patch release here would still need a release in eslint-plugin-jsx-a11y. It's also possible this is already fixed in v3.0. |
Pretty sure this is fixed by #77. eslint-pliugin-react has already been updated with it, and eslint-plugin-jsx-a11y will get it on the next release. |
I use
@babel/plugin-proposal-optional-chaining
like this:I found only
href
will throw error. Other props will not throw error, like this:The text was updated successfully, but these errors were encountered: