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

Upgrade to ESLint 9.x #73

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Upgrade to ESLint 9.x #73

wants to merge 10 commits into from

Conversation

oliverroick
Copy link
Collaborator

@oliverroick oliverroick commented Nov 8, 2024

Resolves #31

  • Upgrade dependencies latest versions:
    • ESlint to 9.14.0
    • eslint-plugin-prettier to 5.2.1
    • eslint-plugin-react to ^7.37.2
    • eslint-plugin-unused-imports to 4.1.4
  • Pin globals package to ^14.0.0, which is required to run ESLint 9.x successfully
  • Add new ESlint config (eslint.config.mjs) compatible with ESLint 9.x
  • Remove previous ESLint config (.eslintrc.js)
  • Remove ESLint from pre-commit
  • Add ESLint Gtihub action
  • Regenerate package-lock to remove some references to a private registry

@oliverroick oliverroick requested a review from yuvipanda November 8, 2024 02:12
@oliverroick oliverroick removed the request for review from yuvipanda November 8, 2024 02:39
@yuvipanda
Copy link
Member

The pre-commit failure is why we couldn't do this last time either I think

@yuvipanda
Copy link
Member

Is it possible to get autofix with this setup too?

@oliverroick
Copy link
Collaborator Author

The auto fix is still running via prettier on pre-commit. The change here only affects the lint check.

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.

Update eslint config for eslint v9.2.0
2 participants