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

eslint: switch to flat configuration #5282

Open
BacLuc opened this issue Jun 2, 2024 · 2 comments
Open

eslint: switch to flat configuration #5282

BacLuc opened this issue Jun 2, 2024 · 2 comments

Comments

@BacLuc
Copy link
Contributor

BacLuc commented Jun 2, 2024

Eslint 9 uses a new configuration format by default (called "flat configuration"). So this PR would need some config changes before we can upgrade to eslint9.

For Nuxt, some further details are described here:
https://eslint.nuxt.com/guide/faq#package-disambiguation
https://eslint.nuxt.com/packages/module

And here is a eslint migration guide for the new config format:
https://eslint.org/docs/latest/use/configure/migration-guide

Originally posted by @usu in #5155 (comment)

Also, we shouldn't need to add @typescript-eslint/eslint-plugin to the dev dependencies anymore.

@BacLuc BacLuc added Meeting Discuss Am nächsten Core-Meeting besprechen type: Frontend type: Print labels Jun 2, 2024
@BacLuc
Copy link
Contributor Author

BacLuc commented Jun 2, 2024

Proposal to resolve this:

  1. switch to new format for all modules
    I would do this first, because i did not get the nuxt eslint helper modules to support both
  2. Update to eslint 9

@manuelmeister
Copy link
Member

Core Meeting Decision

  1. We individually switch the format for each module
  2. Update to eslint 9

@manuelmeister manuelmeister removed the Meeting Discuss Am nächsten Core-Meeting besprechen label Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants