Skip to content
This repository has been archived by the owner on Jan 26, 2019. It is now read-only.

(After ejecting) Error while running ESLint: Cannot find module 'eslint-config-react-app' #338

Open
aurerua opened this issue Jun 11, 2018 · 3 comments

Comments

@aurerua
Copy link
Contributor

aurerua commented Jun 11, 2018

Is this a bug report?

Yes

Can you also reproduce the problem with npm 4.x?

--

Which terms did you search for in User Guide?

eject, eslint

Environment

  1. npm ls react-scripts-ts (if you haven’t ejected):
  2. node -v: v8.9.4
  3. npm -v: 6.0.0
  4. yarn --version (if you use Yarn): 1.7.0
  5. npm ls react-scripts-ts (if you haven’t ejected):

Then, specify:

  1. Operating system: linux (Pop!_OS)
  2. Browser and version (if relevant):

Steps to Reproduce

npx create-react-app my-app --scripts-version=react-scripts-ts
cd my-app
npm run eject
  1. Open a JS file, e.g. ./scripts/start.js
  2. I am using Atom editor with linter-eslint, so it picks up the eslintConfig from ./package.json and it throws the Error: Error while running ESLint: Cannot find module 'eslint-config-react-app'

Expected Behavior

No ESLint Error on opening a JS file.

Which would probably be solved by either have no eslintConfig in ./package.json or have the stated react-app dependency installed.

Actual Behavior

After ejecting, if one then scrolls down to the bottom of ./package.json, it shows:

{
  "eslintConfig": {
    "extends": "react-app"
  }
}

Yet the dependency is missing:

npm ls eslint-config-react-app
my-app@0.1.0 /home/aurelien/tmp/my-app
└── (empty)

Reproducible Demo

--

@wmonk
Copy link
Owner

wmonk commented Jun 11, 2018 via email

@aurerua
Copy link
Contributor Author

aurerua commented Jun 28, 2018

I've created a PR: fix: do not add ESLint config to ejected app #351

I hope this is welcome.

@paulmelnikow
Copy link

Looks like this is fixed!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants