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
Add Support for RegExp match indices #14640
Comments
Since acorn has been released, I would like to specify a new acorn in the espree dependency and make changes to allow 2022 for ecmaVersion. Is this acceptable? https://github.com/acornjs/acorn/blob/master/acorn/CHANGELOG.md#830-2021-05-31 When I make changes to ecmaVersion to allow 2022, acorn also tries to parse class fileds, but espree doesn't yet support it. Is es2022 syntax support done in v8.0.0? |
I saw the meeting memo. My understanding is that support for the ES2022 syntax will not be added to ESLint v7. |
Yes, we will be waiting to add ES2022 syntax in ESLint v8.0.0. This PR updates Acorn in Espree: eslint/espree#505 |
update: regexpp v3.2.0 was released! |
Thank you for letting me know! |
* Update: support new regex d flag (fixes #14640) * Upgrade regexpp * Update lib/rules/no-invalid-regexp.js Co-authored-by: Jordan Harband <ljharb@gmail.com> * Upgrade espree Co-authored-by: Jordan Harband <ljharb@gmail.com> Co-authored-by: Nicholas C. Zakas <nicholas@nczconsulting.com>
The version of ESLint you are using.
7.27.0
The problem you want to solve.
Support RegExp match indices officially, as it has reached stage-4.
refs:
https://github.com/tc39/proposals/blob/master/finished-proposals.md
https://github.com/tc39/proposal-regexp-match-indices
Your take on the correct solution to problem.
regexpp
update: Supports ES2022 RegExp Match Indices mysticatea/regexpp#22espree
update: Update: Support top-level await and regexp match indices espree#505eslint
update: Breaking: support new regex d flag (fixes #14640) #14653We probably need to add
d
to thevalidFlags
of the no-invalid-regexp rule.We probably need to add
d
to theregex
variable of the no-empty-character-class rule.Are you willing to submit a pull request to implement this change?
I will submit a pull request.
The text was updated successfully, but these errors were encountered: