npm-groovy-lint - npm Package Compare versions

Comparing version 10.0.2 to 10.0.3

CHANGELOG.md

@@ -5,2 +5,7 @@ # Changelog
 
+## [10.0.3] 2022-08-15
+
+- Do not output results summary in console logs when output is json or sarif
+- Add test methods for SARIF called by CLI
+
 ## [10.0.2] 2022-08-15
@@ -7,0 +12,0 @@

lib/groovy-lint.js

@@ -462,8 +462,8 @@ // Imports
                 : this.options.failonerror
-                ? "error"
-                : this.options.failonwarning
-                ? "warning"
-                : this.options.failoninfo
-                ? "info"
-                : "none";
+                    ? "error"
+                    : this.options.failonwarning
+                        ? "warning"
+                        : this.options.failoninfo
+                            ? "info"
+                            : "none";
         if (failureLevel === "none") {
@@ -479,3 +479,5 @@ return;
         if (failureLevel === "error" && errorNb > 0) {
-            console.error(`Failure: ${this.lintResult.summary.totalFoundErrorNumber} error(s) have been found`);
+            if (!["json", "sarif"].includes(this.outputType)) {
+                console.error(`Failure: ${this.lintResult.summary.totalFoundErrorNumber} error(s) have been found`);
+            }
             this.status = 1;
@@ -485,5 +487,7 @@ }
         else if (failureLevel === "warning" && (errorNb > 0 || warningNb > 0)) {
-            console.error(
-                `Failure: ${this.lintResult.summary.totalFoundErrorNumber} error(s) have been found \n ${this.lintResult.summary.totalFoundWarningNumber} warning(s) have been found`
-            );
+            if (!["json", "sarif"].includes(this.outputType)) {
+                console.error(
+                    `Failure: ${this.lintResult.summary.totalFoundErrorNumber} error(s) have been found \n ${this.lintResult.summary.totalFoundWarningNumber} warning(s) have been found`
+                );
+            }
             this.status = 1;
@@ -493,5 +497,7 @@ }
         else if (failureLevel === "info" && (errorNb > 0 || warningNb > 0 || infoNb > 0)) {
-            console.error(
-                `Failure: ${this.lintResult.summary.totalFoundErrorNumber} error(s) have been found \n ${this.lintResult.summary.totalFoundWarningNumber} warning(s) have been found \n ${this.lintResult.summary.totalFoundInfoNumber} info(s) have been found`
-            );
+            if (!["json", "sarif"].includes(this.outputType)) {
+                console.error(
+                    `Failure: ${this.lintResult.summary.totalFoundErrorNumber} error(s) have been found \n ${this.lintResult.summary.totalFoundWarningNumber} warning(s) have been found \n ${this.lintResult.summary.totalFoundInfoNumber} info(s) have been found`
+                );
+            }
             this.status = 1;
@@ -498,0 +504,0 @@ }

package.json

 {
   "name": "npm-groovy-lint",
-  "version": "10.0.2",
+  "version": "10.0.3",
   "description": "Lint, format and auto-fix your Groovy / Jenkinsfile / Gradle files",
@@ -5,0 +5,0 @@ "main": "index.js",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

16991508

Lines of code

7261

increased by0.08%

No dependency changes