Skip to content

Commit

Permalink
PA-12336 package update / fail msg
Browse files Browse the repository at this point in the history
  • Loading branch information
SOOS-GSteen committed Dec 14, 2023
1 parent 49410da commit b198be1
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 6 deletions.
8 changes: 4 additions & 4 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
},
"homepage": "https://github.com/soos-io/soos-sast#readme",
"dependencies": {
"@soos-io/api-client": "^0.2.12",
"@soos-io/api-client": "^0.2.13",
"argparse": "^2.0.1",
"glob": "^10.3.10",
"tslib": "^2.6.2"
Expand Down
3 changes: 2 additions & 1 deletion src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,8 @@ class SOOSSASTAnalysis {
});

const exitWithError = verifyScanStatus(scanStatus);
if (this.args.onFailure === OnFailure.Fail && exitWithError) {
if (exitWithError && this.args.onFailure === OnFailure.Fail) {
soosLogger.warn("Failing the build.");
exit(1);
}
} catch (error) {
Expand Down

0 comments on commit b198be1

Please sign in to comment.