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

Use something other than ESLint's built-in rules for formatting #5

Open
boyum opened this issue Nov 5, 2023 · 4 comments
Open

Use something other than ESLint's built-in rules for formatting #5

boyum opened this issue Nov 5, 2023 · 4 comments

Comments

@boyum
Copy link
Contributor

boyum commented Nov 5, 2023

Earlier this year, it was announced that ESLint 9 will not include formatting rules. In the latest version (eslint/eslint#17696) they have deprecated them and they will soon be gone. This project should be updated to support ESLint 9.

The simplest way to fix this is to use @stylistic/eslint-plugin-js which includes the old formatting rules. https://eslint.style/packages/js

@otacke
Copy link
Contributor

otacke commented Nov 6, 2023

Good surveillance of the dev scene!

H5P Group is going to monitor how Stylistic evolves in terms of community based maintenance and use eslint 8 until it's clear what direction makes the most sense for them. They are also considering to create an official configuration on npm then, so I don't think any immediate action needs to be taken here.

@otacke
Copy link
Contributor

otacke commented Nov 6, 2023

Addition: The old rules will not be removed until at least ESlint 10.0.0 (see https://eslint.org/blog/2023/10/deprecating-formatting-rules/), so there really seems to be enough time for assessing the situation.

@boyum
Copy link
Contributor Author

boyum commented Nov 6, 2023

Thanks for the clarification!

@otacke
Copy link
Contributor

otacke commented Apr 8, 2024

Just a note on top: ESlint 9.0.0 is out, and flat config is set to be the new default (can be overridden by environment variables until ESlint 10.0.0). That means the general configuration will need to be changed a little as well. eslint-plugin-import seems to be close to releasing an update to support ESLint 9, so at least this could be tackled shortly.

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

No branches or pull requests

2 participants