HDDS-15793. Bump ESLint to 9.39.4#10697
Conversation
|
Hi @jshukla01, thanks for this patch, it was long overdue. |
@spacemonkd Thanks for taking time for looking into it. Yes I did run pnpm lint after the changes and below is the comparison between v7 and v9. Attaching the full output of the run. |
spacemonkd
left a comment
There was a problem hiding this comment.
Thanks @jshukla01 for the update.
This LGTM, +2.
Currently we do not have linter checks on JS, so we can handle these violations later on in a separate ticket. Would you like to raise an Apache JIRA and attach the error log, maybe you or someone from the community can pick it up later on.
|
Thanks @jshukla01 for the patch. HDDS-15647 introduced conflicts, can you please resolve them? |
Bump eslint version to v9.39.4.
Please describe your PR in detail:
Bumping the eslint version. From 9.0.0 onwards eslint expects flat config files hence added a new file and removed .eslintignore.
Tested it locally by running pnpm list on both v7 and v9 and the result were similar.