Skip to content

Add report job status step to CI workflow#172

Merged
YooSunYoung merged 7 commits intomainfrom
ci-test
Mar 26, 2026
Merged

Add report job status step to CI workflow#172
YooSunYoung merged 7 commits intomainfrom
ci-test

Conversation

@YooSunYoung
Copy link
Copy Markdown
Member

@YooSunYoung YooSunYoung commented Mar 6, 2026

With the matrix strategy, we couldn't use them for branch protection rules.
Here I added the final report job that fails when formatting, test or docs job fails, and not skipped.

We can use this one for the branch protection rule maybe...?

I tested the failure report in #173

@YooSunYoung YooSunYoung changed the title [NO MERGE] Add report job status step to CI workflow Add report job status step to CI workflow Mar 6, 2026
@YooSunYoung YooSunYoung marked this pull request as ready for review March 6, 2026 11:13
@YooSunYoung
Copy link
Copy Markdown
Member Author

Is it not so useful after all cause we will see failing jobs anyway...?

@nvaytet
Copy link
Copy Markdown
Member

nvaytet commented Mar 25, 2026

I think this is useful 👍

@YooSunYoung YooSunYoung merged commit 53f831b into main Mar 26, 2026
9 checks passed
@YooSunYoung YooSunYoung deleted the ci-test branch March 26, 2026 07:51
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