Webpack warnings trigger build errors when running on CI

(Initial discussion starts here.)

Pull request #944 adds a new behaviour related to CIs - crash the build during CI whenever linter warnings are encountered.

This PR however introduced new side effect - webpack warnings will also crash the build, e.g.:

This seems to be a pre-built javascript file. Though this is possible, it's not recommended. Try to require the original source to get better results.

We could consider this particular warning to be harmless.
Since we have no access to webpack config, we cannot use module.noParse option either.

My initial idea was to check warnings to see if there are any related to eslint-loader.

Warnings emitted by eslint-loader contain full loader path, e.g. /long/path/to/node_modules/eslint-loader/index.js.

I couldn't find anything else that would mention eslint or eslint-loader that could be also reliably used to determine that a specific warning comes from ESLint.

Agree. I’m not sure about this specific fix but please send a PR to get the ball rolling.
Then we can consider alternatives.

I dug a bit further and couldn't find anything that would provide information about which loader emitted the warning in a reliable manner.

For example an ESLint warning about eqeqeq, loaders property looks like this:

[
  '/system/path/node_modules/babel-loader/index.js?{"babelrc":false,"presets":["/system/path/node_modules/babel-preset-react-app/index.js"]}',
  '/system/path/node_modules/eslint-loader/index.js'
]

There's also issuer and request which are just long ! webpack strings formed from the info above.

edit. Maybe we could use failOnError and failOnWarning provided by eslint-loader.

Same issue here. My temporary workaround was to unset CI variable in the build script (using .gitlab-ci.yml):

#...

build-app:
  stage: build-app
  cache:
    paths:
      - node_modules
    key: "node_modules"
    untracked: true
  script:
    - npm install
    - unset CI # <-------- this line helped
    - npm run build
  artifacts:
    paths:
      - build

#...

@kachkaev

@kachkaev Are there any side effects? Because of this, I'm still stuck at 0.7.5.

@scholtzm

@scholtzm I have not noticed any. Please share yours – others (including myself) may face them in the next project.

Maybe we can ask Webpack to expose a way to turn that warning off? I never found it useful.

Please see past discussions when people repeatedly asked to treat warnings as errors in CI. This is common practice.

@scholtzm

@scholtzm Can you check if the issue reproduces on master?

I will report back this weekend.

fwiw, I've been getting the initial This seems to be a pre-built javascript file warning for some time, but I just upgraded to 1.0.x and it's being suppressed correctly now. ✨

The funny thing is we didn’t actually do anything to suppress it. I have no idea why it’s gone 😄

I’ll close for now but if somebody can reproduce please give me instructions.

Haha, MYSTERY FIX

I'll keep an eye out on my end for any weird changes here regardless.

Navigation Menu

Repository navigation

Webpack warnings trigger build errors when running on CI #1150

9 remaining items

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Webpack warnings trigger build errors when running on CI #1150

Description

Activity

scholtzm commented on Dec 4, 2016

gaearon commented on Dec 5, 2016

scholtzm commented on Dec 5, 2016

kachkaev commented on Dec 18, 2016

scholtzm commented on Feb 11, 2017

kachkaev commented on Feb 11, 2017

gaearon commented on Feb 11, 2017

9 remaining items

gaearon commented on May 18, 2017

gaearon commented on May 18, 2017

scholtzm commented on May 18, 2017

holman commented on May 22, 2017

gaearon commented on May 22, 2017

gaearon commented on May 22, 2017

holman commented on May 22, 2017

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions