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

Can't create config with React + Typescript + Airbnb #35

Closed
goerlitz opened this issue Aug 16, 2022 · 2 comments · Fixed by #38
Closed

Can't create config with React + Typescript + Airbnb #35

goerlitz opened this issue Aug 16, 2022 · 2 comments · Fixed by #38

Comments

@goerlitz
Copy link

It used to be possible to configure eslint with React + TypeScript + Airbnb style.

Unfortunately, with #33 Airbnb and Google style was removed.

The PR states that TypeScript is not officially supported by Airbnb and Google. But there is actually eslint-config-airbnb-typescript to the rescue.

The current situation leave me clueless how to configure eslint with React + TypeScript + Airbnb correctly.
Should I

  1. configure eslint with React + Typescript and then manually install Airbnb style with Typescript support, or
  2. configure eslint with React + Airbnb style and then manually add Typescript

In either case, I don't really know how to modify .eslintrc appropriately to incorporate the manually installed packages.

I would highly appreciated if this config combination would be supported.

@aladdin-add
Copy link
Member

aladdin-add commented Aug 16, 2022

you can follow the intro in the eslint-config-Airbnb-typescript.

we have a plan to support --config(not implemented yet), so you will be able to easily run npm init @eslint/config --config airbnb-typescript to do so. related: #23

@github-actions
Copy link
Contributor

Oops! It looks like we lost track of this issue. What do we want to do here? This issue will auto-close in 7 days without an update.

@github-actions github-actions bot added the Stale label Oct 15, 2022
@mdjermanovic mdjermanovic linked a pull request Oct 16, 2022 that will close this issue
@nzakas nzakas moved this to Complete in Triage Jan 3, 2023
@nzakas nzakas added this to Triage Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants