Set comment as output

Move the logic from the caller as a negation instead of trying to
negate it by hand.

dist/index.js

@@ -60,9 +60,9 @@ function commentPr(summary, config) {
     return __awaiter(this, void 0, void 0, function* () {
         const commentContent = summary.stringify();
         core.setOutput('comment-content', commentContent);
-        if (config.comment_summary_in_pr !== 'always' &&
-            config.comment_summary_in_pr === 'on-failure' &&
-            process.exitCode !== core.ExitCode.Failure) {
+        if (!(config.comment_summary_in_pr === 'always' ||
+            (config.comment_summary_in_pr === 'on-failure' &&
+                process.exitCode === core.ExitCode.Failure))) {
             return;
         }
         if (!github.context.payload.pull_request) {

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "dependency-review-action",
-  "version": "4.1.2",
+  "version": "4.1.3",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "dependency-review-action",
-      "version": "4.1.2",
+      "version": "4.1.3",
       "license": "MIT",
       "dependencies": {
         "@actions/core": "^1.10.1",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "dependency-review-action",
-  "version": "4.1.2",
+  "version": "4.1.3",
   "private": true,
   "description": "A GitHub Action for Dependency Review",
   "main": "lib/main.js",

src/comment-pr.ts

@@ -22,9 +22,11 @@ export async function commentPr(
   core.setOutput('comment-content', commentContent)
 
   if (
-    config.comment_summary_in_pr !== 'always' &&
-    config.comment_summary_in_pr === 'on-failure' &&
-    process.exitCode !== core.ExitCode.Failure
+    !(
+      config.comment_summary_in_pr === 'always' ||
+      (config.comment_summary_in_pr === 'on-failure' &&
+        process.exitCode === core.ExitCode.Failure)
+    )
   ) {
     return
   }