don't set output on every run

Author: Federico Builes

dist/index.js

@@ -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",

dist/index.js.map

@@ -3,7 +3,6 @@ import * as core from '@actions/core'
 import * as githubUtils from '@actions/github/lib/utils'
 import * as retry from '@octokit/plugin-retry'
 import {RequestError} from '@octokit/request-error'
-import {ConfigurationOptions} from './schemas'
 
 const retryingOctokit = githubUtils.GitHub.plugin(retry.retry)
 const octo = new retryingOctokit(
@@ -13,22 +12,11 @@ const octo = new retryingOctokit(
 // Comment Marker to identify an existing comment to update, so we don't spam the PR with comments
 const COMMENT_MARKER = '<!-- dependency-review-pr-comment-marker -->'
 
-export async function commentPr(
-  summary: typeof core.summary,
-  config: ConfigurationOptions
-): Promise<void> {
+export async function commentPr(summary: typeof core.summary): Promise<void> {
   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
-  ) {
-    return
-  }
-
   if (!github.context.payload.pull_request) {
     core.warning(
       'Not in the context of a pull request. Skipping comment creation.'

package-lock.json

@@ -144,7 +144,13 @@ async function run(): Promise<void> {
 
     summary.addScannedDependencies(changes)
     printScannedDependencies(changes)
-    await commentPr(core.summary, config)
+    if (
+      config.comment_summary_in_pr === 'always' ||
+      (config.comment_summary_in_pr === 'on-failure' &&
+        process.exitCode === core.ExitCode.Failure)
+    ) {
+      await commentPr(core.summary)
+    }
   } catch (error) {
     if (error instanceof RequestError && error.status === 404) {
       core.setFailed(