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

feat: compatible with eslint-webpack-plugin #1

Merged
merged 1 commit into from
Dec 10, 2020

Conversation

fangbinwei
Copy link
Collaborator

  1. compatible with eslint-webpack-plugin. it seems this plugin groups eslint-loader's errors as Module Error before. So now grouping eslint-webpack-plugin's error as transformers/esLintError.js at least.

eslint-loader will emit multi-errors according to the number of files with errors. eslint-webpack-plugin will gather and throw only one compilation.errors/warnings. It is caused by the difference between loader and plugin in some degree.

  1. change 1 errors to 1 error

when using eslint-loader:
image

When using eslint-webpack-plugin before this pr:
image

When using eslint-webpack-plugin after this pr: :

image

Maybe this plugin is outdated, I'm not sure if there is a good substitute.

@haoqunjiang haoqunjiang merged commit 60dcf38 into haoqunjiang:master Dec 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants