Patch for exit code when only warnings are detected (#28)

Fixes #25

Author: tclindner

CHANGELOG.md

@@ -11,6 +11,10 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Removed
 
+## [2.1.1] - 2017-04-10
+### Fixed
+- CLI so process exit code remains 0 if only warnings are detected
+
 ## [2.1.0] - 2017-04-09
 ### Changed
 - CLI process exit code to 2 (non-zero) if issues are detected in the scan

README.md

@@ -48,6 +48,7 @@ First thing first, let's make sure you have the necessary pre-requisites.
 | pjl-cli --file <file path> | -f | File path including name. Defaults to package.json |
 | pjl-cli --rule <rule name> | -r | Valid rule name to check. Defaults to nothing |
 | pjl-cli --rules-file <file path> | -c | File path of .npmpackagejsonlintrc |
+| pjl-cli --rule-severity <rule severity> | -s | "error" or "warning". Defaults to "error" |
 | pjl-cli --ignore-warnings | -w | Ignore warnings |
 
 ### Examples

package.json

@@ -1,6 +1,6 @@
 {
   "name": "npm-package-json-lint",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "description": "CLI app for linting package.json files.",
   "keywords": [
     "lint",

src/cli.js

@@ -94,10 +94,11 @@ try {
   for (const issueType in output) {
     const issues = output[issueType];
 
-    if (issues.length > noIssues) {
+    if (issues.length > noIssues && issueType === 'errors') {
       exitCode = issuesDetectedErrorCode;
-      reporter.write(output[issueType], issueType);
     }
+
+    reporter.write(output[issueType], issueType);
   }
 
   const formattedFileName = chalk.bold.green(filePath);