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

[427] Migrate ESLint config to flat format #439

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

klappradla
Copy link
Member

Changes:

  • Run ESLint migration task
  • Add the missing dependencies for new ESLint config
  • Remove ignore-path CLI flag
  • Remove --ext CLI flag
  • Update eslint packages for typescript
  • Fix wrong reporting of unused-var
  • Make new config compatible and remove old

Following the migration guide at https://eslint.org/docs/latest/use/configure/migration-guide#--ext

Running the migration task: `npx @eslint/migrate-config .eslintrc.json`
This was suggested by the migration task:

`yarn add -D @eslint/compat globals @eslint/js @eslint/eslintrc`
The flag is deprecated and does not work any more. Instead, using this
approach: https://eslint.org/docs/latest/use/configure/ignore#including-gitignore-files
Follow the docs for typescript: https://typescript-eslint.io/rules/no-unused-vars/

tl;dr: Disable the rule from ESLint and use the one from
ESLint-TypeScript.
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.

1 participant