Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Use caching with eslint-loader and ESLint CLI #954

Closed
edmorley opened this issue Jun 13, 2018 · 1 comment
Closed

Use caching with eslint-loader and ESLint CLI #954

edmorley opened this issue Jun 13, 2018 · 1 comment
Assignees
Milestone

Comments

@edmorley
Copy link
Member

Trying Neutrino master with mozilla/treeherder, the prod warm cache build times are currently:

  • no lint preset: 34s
  • @neutrinojs/airbnb: 46s
  • @neutrinojs/airbnb + eslint-loader's cache: true: 34s

It seems like an easy win.

https://github.com/webpack-contrib/eslint-loader#cache-default-false

@edmorley edmorley added this to the v9 milestone Jun 13, 2018
@edmorley edmorley self-assigned this Jun 13, 2018
@edmorley edmorley changed the title Use caching with eslint-loader Use caching with eslint-loader and ESLint CLI Jun 13, 2018
@edmorley
Copy link
Member Author

Similarly, using the --cache option with ESLint's CLI, has the following effect:

  • mozilla/treeherder: 12.7s -> 5.2s
  • mozilla-neutrino/neutrino-dev: 7.6s -> 3.5s

https://eslint.org/docs/user-guide/command-line-interface#caching

edmorley added a commit that referenced this issue Jun 14, 2018
This significantly improves the performance when using ESLint via its
CLI (ie the `yarn lint` command configured by create-project), and
also via `eslint-loader` (when using `yarn start` or `yarn build`):
https://eslint.org/docs/user-guide/command-line-interface#caching
https://github.com/webpack-contrib/eslint-loader#cache-default-false

Fixes #954.
acconrad pushed a commit to acconrad/neutrino-dev that referenced this issue Jul 13, 2018
This significantly improves the performance when using ESLint via its
CLI (ie the `yarn lint` command configured by create-project), and
also via `eslint-loader` (when using `yarn start` or `yarn build`):
https://eslint.org/docs/user-guide/command-line-interface#caching
https://github.com/webpack-contrib/eslint-loader#cache-default-false

Fixes neutrinojs#954.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

1 participant