[ERROR] npm info ok ==> wrong Log Level

I’m very confused, when I see the following log message in my console. What is it? Error or INFO? Please change the Log-Level

[INFO] --- frontend-maven-plugin:1.4:npm (npm info) @ XYZ ---
[INFO] Running 'npm info npm version --always-auth=false --loglevel=info --progress=false --https-proxy=http://localhost:3128 --proxy=http://localhost:3128' in C:\Users\XYZ\XYZ
[ERROR] npm info it worked if it ends with ok
[ERROR] npm info using npm@3.10.10
[ERROR] npm info using node@v6.11.0
[ERROR] npm info attempt registry request try #1 at 09:19:45
[ERROR] npm http request GET http://MY-NPM-REPO/repository/npm-group/npm
[ERROR] npm http 200 http://MY-NPM-REPO/repository/npm-group/npm
[INFO] 6.1.0
[ERROR] npm info ok

[just-paja]

Seen #411.

Sorry to throw links at you, but please see wiki on stderr.

Standard error is another output stream typically used by programs to output error messages or diagnostics.

npm is correctly dumping diagnostic information to stderr. I realize that defining diagnostic information is debatable, but let’s not go there. The point is that not everything in stderr is error.

Because we’re never going to be able to change how maven and npm interpret usage of stdout and stderr, we should treat this plugin as mediator between the two and treat npm’s stderr as diagnostic output, not errors. Maven actually works in simillar way. Check out that all the [INFO]s and [ERROR]s are sent to stdout.

I dare to say that this:

eirslett: At least as far as this plugin is concerned, all it does is forward stdout to Maven’s stdout, and stderr to Maven’s stderr. I’d say it’s a feature, not a bug.

is the wrong approach.

There is one ridiculous example, output of running jest:

[ERROR] PASS test src/reducers/__tests__/forms.js

[eirslett]

Should be fixed in the latest release, I think?