Merge pull request #516 from mriehema/fix-515

Fix 515 - change yarn warnings in log
eirslett · Nov 15, 2016 · 22d08be · 22d08be
2 parents 5ccbec2 + 4919bc1
commit 22d08be
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,7 +2,12 @@
 
 ## Changelog
 
-### 1.2-SNAPSHOT
+### 1.3
+
+* Fix `yarn` for Windows
+* Fix #515: Change "yarn warning" from ERROR to WARNING in log
+
+### 1.2
 
 * New goals `yarn` and `install-node-and-yarn` for Yarn support 
 

diff --git a/...core/src/main/java/com/github/eirslett/maven/plugins/frontend/lib/InputStreamHandler.java b/...core/src/main/java/com/github/eirslett/maven/plugins/frontend/lib/InputStreamHandler.java
@@ -53,8 +53,8 @@ private static LogLevelAgnosticLogger errorLoggerFor(final Logger logger){
         return new LogLevelAgnosticLogger() {
             @Override
             public void log(String value) {
-                // fix #343
-                if (value.startsWith("npm WARN ")) {
+                // fix #343 and #515
+                if (value.startsWith("npm WARN ") || value.startsWith("warning ")) {
                     logger.warn(value);
                 } else {
                     logger.error(value);