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

Update JavaScript / ESLint guidance #309

Closed
coliff opened this issue Apr 11, 2024 · 0 comments · Fixed by #310
Closed

Update JavaScript / ESLint guidance #309

coliff opened this issue Apr 11, 2024 · 0 comments · Fixed by #310
Assignees
Labels
2 2 days documentation Documentation additions, updates and corrections.

Comments

@coliff
Copy link
Member

coliff commented Apr 11, 2024

ESlint released a major new version (v9) with many big changes... including a new config file format (the old .eslintrc config is now deprecated).

I think we should make a PR to the Devguide update our guidelines to match this since the new flat config is also compatible with ESLint v8.

I also recommended changing our base config to use eslint-recommended and not Google's which is now deprecated and hasn't been updated in 5 years (I just discovered this today!)

@coliff coliff added 2 2 days documentation Documentation additions, updates and corrections. labels Apr 11, 2024
@coliff coliff self-assigned this Apr 12, 2024
@coliff coliff linked a pull request Apr 12, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2 2 days documentation Documentation additions, updates and corrections.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant