Run Lint from the command line?

[danielstern]

It would be great to be able to type
eslint into bash and have it run ESLint just like it was building. As it is, it can't find the config file unless I eject. Is ejection really necessary here?

[gaearon]

What is your use case for running lint separately?

[danielstern]

We are running it in coordination with Husky(https://github.com/typicode/husky) to run ESLint in our pre-push.
We're hoping for something like

  "scripts": {
    "prepush": "set CI=true&&npm test&&eslint",
  }

[caesarsol]

+1, could be useful to lint in CI if someone has no tests.
Plus, someone does a complete lint before committing or as a precommit hook.

[sakulstra]

even if one has tests, this would be useful. In our projects we usually first check if linting is correct before we actually run the tests in another ci step.

Is this currently possible with create react app? e.g. sth like:

"scripts": {
"lint": "react-scripts run lint"
"test": "react-scripts run test"
}

[caesarsol]

Yes, I erronously thought linting was checked inside the testing.

A workaround is possible (npm >= 3), by directly executing node node_modules/eslint/eslint.js fron npm scripts, pointing it to the right eslint react-app config.

(I'm currently linking that path to node_modules/.bin/eslint as a dirty trick to lint from Sublime Text)

[danielstern]

@caesarsol Interesting, an all-purpose solution would be great

[gaearon]

We are running it in coordination with Husky(https://github.com/typicode/husky) to run ESLint in our pre-push.

Why not build the project too? It could uncover other critical issues that ESLint might miss.
And running npm run build includes linting.

[talkingtab]

My use case: because my component interacts with the server, and because I need the real server, I end up having to run npm build (which slowly builds an optimized package) when it would be very nice to just say "run eslint" and find out quickly whether I have syntax errors.

I have tried the proxy web server but because my application requires an https connection and because the interaction between the server and client is complex I have never found a way to have that work successfullly. This is an Oauth2 oriented project.

[stevenmusumeche]

+1

[Andrei-Vakulski]

+1, basicly want to integrate a https://github.com/observing/pre-commit hook to lint my app.

[Morkowski]

@caesarsol Is your solution works with new version CRA? Tried executing:

node node_modules/eslint/bin/eslint.js --config node_modules/eslint-config-react-app/index.js App.js

but I can't see any output. What I'm doing wrong?

[talkingtab]

maybe App.js does not have any errors? Try inserting and 'x' some place where it will cause an error ...

[Morkowski]

@talkingtab: I tried with incorrect code in App.js file, which shows errors in browser box while running CRA.