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

fix(eslint): invalidate the cache when .eslintignore changes #5455

Merged
merged 1 commit into from May 11, 2020
Merged

fix(eslint): invalidate the cache when .eslintignore changes #5455

merged 1 commit into from May 11, 2020

Conversation

godkun
Copy link
Contributor

@godkun godkun commented May 8, 2020

add the .eslintignore file to the cache generation algorithm

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Docs
  • Underlying tools
  • [] Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

Other information:

eslint-loader 运行时会生成 cache ,再次运行时,会检查 cache 是否可用

理论上 .eslintignore 发生变化时, cache 应该失效,重新运行

@vue/cli-plugin-eslint 没有把该文件加入 cache 的生成算法中

从而导致了当 .eslintignore 发生变化时,cache没有失效,eslint 忽略规则没有生效

所以需要将 .eslintignore 文件加入到 cache 处理中

add the .eslintignore file to the cache generation algorithm
@sodatea sodatea changed the title fix(cli-plugin-eslint) fix(eslint): invalidate the cache when .eslintignore changes May 11, 2020
@sodatea sodatea merged commit a91649e into vuejs:dev May 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants