Skip to content

[failOnError] show eslint errors when failOnError is disabled #85

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Conversation

GertSallaerts
Copy link
Contributor

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

Eslint errors are no longer shown when you disable failOnError. See #84

Breaking Changes

Additional Info

@codecov
Copy link

codecov bot commented Feb 16, 2021

Codecov Report

Merging #85 (72c52f0) into master (4930a6f) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #85   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         8           
  Lines          251       253    +2     
  Branches        71        72    +1     
=========================================
+ Hits           251       253    +2     
Impacted Files Coverage Δ
src/index.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4930a6f...72c52f0. Read the comment docs.

@GertSallaerts GertSallaerts force-pushed the fix/fail-on-error-display branch 2 times, most recently from 603ea35 to 78b79d0 Compare February 16, 2021 13:53
@GertSallaerts GertSallaerts force-pushed the fix/fail-on-error-display branch from 78b79d0 to 72c52f0 Compare February 16, 2021 20:59
@ricardogobbosouza
Copy link
Collaborator

Hi @GertSallaerts
You need signed the CLA

@ricardogobbosouza
Copy link
Collaborator

Ahh...thanks for PR 😄

@ricardogobbosouza ricardogobbosouza self-requested a review February 16, 2021 21:02
@ricardogobbosouza ricardogobbosouza merged commit 69be533 into webpack-contrib:master Feb 17, 2021
@ZLevine
Copy link

ZLevine commented Feb 18, 2021

Thanks for your quick work on this one @GertSallaerts and @ricardogobbosouza 🚀 Any idea when this will be released?

@ricardogobbosouza
Copy link
Collaborator

Hi @ZLevine
Will be released today =)

@ZLevine
Copy link

ZLevine commented Feb 18, 2021

@ricardogobbosouza thanks! I see it now :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants