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

Remove unmaintined airbnb and flowtype eslint plugins #3970

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

jcoyne
Copy link
Collaborator

@jcoyne jcoyne commented Oct 30, 2024

Theae prevent an upgrade to eslint 9. Eslint < 9 is EOL.

@cbeer
Copy link
Collaborator

cbeer commented Oct 31, 2024

👎 . Removing the airbnb defaults removes something like 2/3 of the style rules we want to enforce. While we could bring them in, I think it's also ok to wait for them to update upstream.

@jcoyne
Copy link
Collaborator Author

jcoyne commented Oct 31, 2024

@cbeer there's no indication that airbnb is going to continue to maintain it. It was last published 3 years ago. It's last commit was a year ago. Eslint version 8 is past it's EOL https://eslint.org/version-support/

@marlo-longley
Copy link
Member

I am running into this issue now. There is a comment from Nov 2024 saying the airbnb plugin is still being worked on?

airbnb/javascript#2961

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.

3 participants