Skip to content

ci: use test gate job as required job#896

Merged
fi3ework merged 1 commit intomainfrom
ci-gate
Jan 22, 2026
Merged

ci: use test gate job as required job#896
fi3ework merged 1 commit intomainfrom
ci-gate

Conversation

@fi3ework
Copy link
Copy Markdown
Member

Summary

using a gate job as the required job in ruleset, to prevent forever hang required job in doc only changed PR (since no test job will run e.g. #895)

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings January 22, 2026 07:54
@fi3ework fi3ework changed the title chore: use test gate job as required job ci: use test gate job as required job Jan 22, 2026
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying rstest with  Cloudflare Pages  Cloudflare Pages

Latest commit: 22a82af
Status: ✅  Deploy successful!
Preview URL: https://ce75d5c7.rstest.pages.dev
Branch Preview URL: https://ci-gate.rstest.pages.dev

View logs

Copy link
Copy Markdown
Contributor

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 adds a gate job to the test workflow to prevent required jobs from hanging indefinitely when only documentation files are changed. The gate job always runs and conditionally checks test results based on whether tests should have run.

Changes:

  • Added a test-gate job that acts as a gatekeeper for the test workflow
  • The gate job depends on prepare, ut, and e2e jobs and always runs
  • Conditionally validates that tests passed when non-documentation files were changed or on scheduled runs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@fi3ework fi3ework merged commit 652dec3 into main Jan 22, 2026
18 checks passed
@fi3ework fi3ework deleted the ci-gate branch January 22, 2026 08:11
@9aoy 9aoy mentioned this pull request Jan 27, 2026
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