-
-
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
Support ChainExpression
AST node type for optional chaining
#103
Comments
See #102. |
zburke
added a commit
to folio-org/ui-data-import
that referenced
this issue
Oct 11, 2020
Due to a [bug in jsx-exlint](jsx-eslint/jsx-ast-utils#103), the changed line causes ESLint to die. The bug [has been fixed](jsx-eslint/jsx-ast-utils#102) but hasn't made it's way into a release yet, and it would be really nice to have lint working again! Refs UIDATIMP-690
zburke
added a commit
to folio-org/ui-data-import
that referenced
this issue
Oct 12, 2020
Due to a [bug in jsx-exlint](jsx-eslint/jsx-ast-utils#103), the changed line causes ESLint to die. The bug [has been fixed](jsx-eslint/jsx-ast-utils#102) but hasn't made its way into a release yet, and it would be really nice to have lint working again! Refs UIDATIMP-690
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Symptom:
Began seeing
The prop value with an expression type of ChainExpression could not be resolved. Please file issue to get this fixed immediately.
in eslint output after upgrading.ECMAscript Optional Chaining shipped, resulting in various changes to parsers and javascript/typescript tools to support it. Initially there were different, non-standard names for the AST nodes to represent it. Over this year, consensus has settled out, including in the parser that eslint uses, estree (see estree/estree#204 ).
jsx-ast-utils should be updated to support the new AST nodes.
Versions:
[email protected]
[email protected] (via [email protected], [email protected])
Minimal test case
Component.jsx
.eslintrc
run
Related:
Example AST (via console.log):
The text was updated successfully, but these errors were encountered: