Skip to content

Conversation

@mjun0812
Copy link
Owner

Fix: PR status check cannot fetch correct status from CI workflow using matrix strategy.

Copilot AI review requested due to automatic review settings June 14, 2025 04:53
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes an issue with the GitHub Actions CI status check when using a matrix strategy by adding a new job to inspect the results of dependent jobs.

  • Introduces a "ci-check" job that depends on the "ci" job.
  • Uses conditional steps to exit with a failure status if any job fails or is cancelled, and with success if all jobs pass or are skipped.
Comments suppressed due to low confidence (1)

.github/workflows/test.yml:71

  • [nitpick] Add a comment explaining the purpose of using 'if: always()' with this job to ensure reviewers understand that it is meant to always run regardless of previous job statuses.
if: always()

@mjun0812 mjun0812 merged commit f03524b into main Jun 14, 2025
5 checks passed
@mjun0812 mjun0812 deleted the fix/CI-status branch June 14, 2025 05:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants