Consider skipped checks as meeting Maestro policy (#3636)

dotnet · Sep 17, 2024 · f8cd36c · f8cd36c
1 parent b6dfd2b
commit f8cd36c
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/Microsoft.DotNet.Darc/DarcLib/GitHubClient.cs b/src/Microsoft.DotNet.Darc/DarcLib/GitHubClient.cs
@@ -1015,7 +1015,7 @@ private async Task<IList<Check>> GetChecksFromChecksApiAsync(string owner, strin
                         CheckStatus.Queued or CheckStatus.InProgress => CheckState.Pending,
                         CheckStatus.Completed => (run.Conclusion?.Value) switch
                         {
-                            CheckConclusion.Success => CheckState.Success,
+                            CheckConclusion.Success or CheckConclusion.Skipped => CheckState.Success,
                             CheckConclusion.ActionRequired or CheckConclusion.Cancelled or CheckConclusion.Failure or CheckConclusion.Neutral or CheckConclusion.TimedOut => CheckState.Failure,
                             _ => CheckState.None,
                         },