Skip to content

Commit a020b9b

Browse files
author
Adam Hofmann
authored
Merge pull request #9 from scribd/adamh/status-patch
Resolving workflow status as successful when all jobs are either successful or skipped
2 parents 3efab95 + 3c5b55d commit a020b9b

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

action.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ runs:
1414
- name: Install required libraries for gems
1515
shell: bash
1616
run: |
17-
gem install octokit
18-
gem install dogapi
17+
gem install octokit --silent
18+
gem install dogapi --silent
1919
- name: Export extra data
2020
shell: bash
2121
run: |

report_github_metrics.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ def collect_job_metrics(job, tags)
2121
def collect_workflow_metrics(jobs, tags)
2222
start = jobs.min_by{|job| job["started_at"]}["started_at"]
2323
finish = jobs.max_by{|job| job["completed_at"]}["completed_at"]
24-
status = jobs.all?{|job| job["conclusion"] == "success"} ? "success" : "failure"
24+
status = jobs.all?{|job| ["success", "skipped"].include? job["conclusion"]} ? "success" : "failure"
2525
[[
2626
"workflow_duration",
2727
finish - start,

0 commit comments

Comments
 (0)