[CI] Do Not Consider a Job Failed if Steps Were Skipped

This patch makes it so that skipped steps do not cause a job to be
considered failed. The windows premerge jobs currently skip the
build/test step if there are no projects to build/test. These show up as
failures in the dashboard even though everything executed perfectly
fine.

Reviewers: lnihlen, Keenuts

Reviewed By: lnihlen

Pull Request: https://github.com/llvm/llvm-project/pull/127279
This commit is contained in:
Aiden Grossman 2025-02-14 19:14:56 -08:00 committed by GitHub
parent 97d2cfeab3
commit d7b89b0dca
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -177,7 +177,7 @@ def get_per_workflow_metrics(
# longer in a testing state and we can directly assert the workflow
# result.
for step in workflow_job.steps:
if step.conclusion != "success":
if step.conclusion != "success" and step.conclusion != "skipped":
job_result = 0
break