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

chore: Update linter so we can leverage modern JS features #1172

Merged
merged 3 commits into from
Sep 11, 2024

Conversation

joe-yeager
Copy link
Contributor

Description and Context

Update the linter to use es2022 since node 18 fully supports es2022

https://node.green/#ES2022

@@ -6,7 +6,7 @@
"private": true,
"devDependencies": {
"depcheck": "1.3.1",
"eslint": "7.32.0",
"eslint": "^8.56.0",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OOC, why this version of eslint? Is it just the latest version from 2022?

Copy link
Contributor Author

@joe-yeager joe-yeager Sep 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good question. V7 didn't seem to have support for 2022, and V9 introduced some fairly large changes to the config files. It dropped support for the .eslintignore, renamed eslintrc.js to eslint.config.js, and significantly changed the format of eslint.config.js.

Also, LDL is using V8

Copy link
Contributor

@kemmerle kemmerle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@joe-yeager joe-yeager merged commit 62d7c91 into main Sep 11, 2024
1 check passed
@joe-yeager joe-yeager deleted the jy/update-linter branch September 11, 2024 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants