Skip to content

feat(eslint): upgrade eslint 9#116

Merged
raphaelpor merged 1 commit intostenciljs:mainfrom
Jorciney:feature/updating-estlint
Nov 2, 2024
Merged

feat(eslint): upgrade eslint 9#116
raphaelpor merged 1 commit intostenciljs:mainfrom
Jorciney:feature/updating-estlint

Conversation

@Jorciney
Copy link
Copy Markdown
Contributor

@Jorciney Jorciney commented Sep 11, 2024

Upgrade ESLint Version

This pull request upgrades ESLint to the latest version, improving compatibility with modern JavaScript features and enhancing overall linting performance.

Checklist

Changes made:

  • Updated the ESLint version in the project dependencies.
  • Updated @typescript-eslint/eslint-plugin & @typescript-eslint/parser
  • As rollup-plugin-node-resolve has been deprecated, I've replaced it with @rollup/plugin-node-resolve
  • Modified configuration files to accommodate any breaking changes introduced in the ESLint upgrade.
  • Ensured all existing linting rules continue to function as expected with the new version.
  • Updated workflow to use node version >= 18

This upgrade should provide better support for newer ECMAScript features and help maintain code quality across the project.

Resolves the following issues:
#114
#104

@Jorciney Jorciney marked this pull request as draft September 11, 2024 16:22
@Jorciney Jorciney marked this pull request as ready for review September 11, 2024 20:46
@Jorciney Jorciney marked this pull request as draft September 11, 2024 20:49
@Jorciney Jorciney force-pushed the feature/updating-estlint branch from 5308379 to b7097c4 Compare September 12, 2024 15:58
@Jorciney Jorciney marked this pull request as ready for review September 12, 2024 16:10
@marcomattes
Copy link
Copy Markdown

Please merge. We urgently want to switch to Eslint 9

@raphaelpor raphaelpor merged commit 8c40fcb into stenciljs:main Nov 2, 2024
@iterox iterox mentioned this pull request Nov 5, 2024
3 tasks
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