no message

brandonpage · Nov 4, 2024 · 5ed6f3e · 5ed6f3e
1 parent 66aa1a9
commit 5ed6f3e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/.github/DangerFiles/StaticAnalysis.rb b/.github/DangerFiles/StaticAnalysis.rb
@@ -25,11 +25,13 @@
     print "file name: #{report_file_name}\n"
 
     if modified_file_names.include?(report_file_name) || added_file_names.include?(report_file_name)
-        print "file match! #{file}"
+        print "file match! #{file}\n"
         issues = report['diagnostics']
+        print "issue count: #{issues.count}\n"
         for i in 0..issues.count-1
             unless issues[i].nil?
-            message << "#{file_path.split('/').last} | #{issues[i]['type']} | #{issues[i]['category']} | #{issues[i]['description']} | #{issues[i]['location']['line']} | #{issues[i]['location']['col']}\n"
+                print "#{file_path.split('/').last} | #{issues[i]['type']} | #{issues[i]['category']} | #{issues[i]['description']} | #{issues[i]['location']['line']} | #{issues[i]['location']['col']}\n"
+                # message << "#{file_path.split('/').last} | #{issues[i]['type']} | #{issues[i]['category']} | #{issues[i]['description']} | #{issues[i]['location']['line']} | #{issues[i]['location']['col']}\n"
             end
         end
     end

diff --git a/.github/workflows/reusable-workflow.yaml b/.github/workflows/reusable-workflow.yaml
@@ -36,7 +36,7 @@ jobs:
           code-coverage: ${{ github.event_name == 'pull_request' }}
           upload-logs: always
       - name: Danger Test Results
-        if: failure()
+        if: (github.event_name == 'pull_request') && failure()
         env:
           DANGER_GITHUB_API_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |