Skip to content

Commit

Permalink
CI summary fix (NVIDIA#3826)
Browse files Browse the repository at this point in the history
* Fix escape sequence in regex.

prepare-execution-summary.py:24: SyntaxWarning: invalid escape sequence '\d'

* Skip jobs that never ran when preparing time summaries.

There will not be an entry in `job_times` if e.g. `job` is a test job that depends on a failed build job.
  • Loading branch information
alliepiper authored Feb 17, 2025
1 parent 498095d commit 33d4f3d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/actions/workflow-results/prepare-execution-summary.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ def natural_sort_key(key):
# Natural sort impl (handles embedded numbers in strings, case insensitive)
return [
(int(text) if text.isdigit() else text.lower())
for text in re.split("(\d+)", key)
for text in re.split("(\\d+)", key)
]


Expand Down Expand Up @@ -69,7 +69,7 @@ def update_summary_entry(entry, job, job_times=None):
else:
entry["failed"] += 1

if job_times:
if job_times and job["id"] in job_times:
time_info = job_times[job["id"]]
job_time = time_info["job_seconds"]
command_time = time_info["command_seconds"]
Expand Down

0 comments on commit 33d4f3d

Please sign in to comment.