Skip to content
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

Bump ESLint ecmaVersion to 2022 #3386

Open
Gazook89 opened this issue Apr 2, 2024 · 3 comments
Open

Bump ESLint ecmaVersion to 2022 #3386

Gazook89 opened this issue Apr 2, 2024 · 3 comments
Labels
dependencies Pull requests that update a dependency file

Comments

@Gazook89
Copy link
Collaborator

Gazook89 commented Apr 2, 2024

Your idea:

Right now our .eslintrc.js config file is set to use ecma version 2021, and I'd like to bump it to 2022. Here is a link that briefly describes some changes: https://dev.to/jasmin/whats-new-in-es2022-1de6

In particular, I like the d regex flag for matching indices. It is used in editor.jsx line 155 right now. Because it is not in ECMA 2021, it throws a lint error, but it works in the browser.

Any objection? Or want to go further, to ECMA 2023? Or we can set the config to use lastest. I don't have any need for anything beyond 2022 (except 2024 may include Temporal when it comes out).

@5e-Cleric 5e-Cleric added the dependencies Pull requests that update a dependency file label May 28, 2024
@dbolack-ab
Copy link
Collaborator

This looks to be resolved by the current config?

@calculuschild
Copy link
Member

If the linter isn't throwing an error anymore we can close this.

@dbolack-ab
Copy link
Collaborator

It's not for me. @Gazook89? ( line number is now 215 )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

No branches or pull requests

4 participants