Skip to content

ci: run the develop CI suite on stacked and draft PRs via a run-ci label - #6275

Merged
avallete merged 3 commits into
developfrom
ci/run-ci-label-stacked-prs
Aug 19, 2026
Merged

ci: run the develop CI suite on stacked and draft PRs via a run-ci label#6275
avallete merged 3 commits into
developfrom
ci/run-ci-label-stacked-prs

Conversation

@avallete

Copy link
Copy Markdown
Member

Summary

Ready PRs targeting develop already get Test, preview CLI packages, and PR-title lint. Stacked PRs (base is another branch) never enter those workflows, and drafts skip the jobs.

This adds a run-ci label that calls the same Test and preview suites as reusable workflows, including while the PR is still a draft. Ready develop PRs stay on the existing workflows so required check names are unchanged.

  • Add run-ci to start the suite; remove it to cancel in-progress opt-in runs.
  • Other labels do not start or cancel Test / preview.
  • Independent of run-live-e2e-ci.
  • After a stacked PR is retargeted onto develop, push or reopen so the native required checks populate.

The run-ci repository label already exists.

Linked issue

N/A — maintainer CI workflow (exempt).

Stacked PRs never hit Test or preview because those workflows filter on develop, and drafts skip the jobs. A dedicated run-ci workflow calls the same suites when the label is present so required develop checks stay unchanged.
@avallete
avallete requested a review from a team as a code owner August 19, 2026 11:28
@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Supabase CLI preview

npx --yes https://pkg.pr.new/supabase/cli/supabase@90bcde725a35098c3778fea24815027fb5a579be

Preview package for commit 90bcde7.

@avallete
avallete enabled auto-merge August 19, 2026 12:00

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: bf7112a841

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/run-ci.yml Outdated
secrets: inherit forwarded every repo secret into PR-controlled Test and preview calls. Those jobs only need the dependency-firewall token, which matches the existing build-cli-artifacts allowlist.
@avallete
avallete marked this pull request as draft August 19, 2026 13:01
auto-merge was automatically disabled August 19, 2026 13:01

Pull request was converted to draft

@avallete avallete added the run-ci Run full develop CI (tests + preview pkgs) on drafts and non-develop PRs. label Aug 19, 2026
@avallete
avallete marked this pull request as ready for review August 19, 2026 13:31
@avallete avallete removed the run-ci Run full develop CI (tests + preview pkgs) on drafts and non-develop PRs. label Aug 19, 2026
@avallete
avallete enabled auto-merge August 19, 2026 13:31
@avallete
avallete added this pull request to the merge queue Aug 19, 2026
Merged via the queue into develop with commit 85df55e Aug 19, 2026
62 of 63 checks passed
@avallete
avallete deleted the ci/run-ci-label-stacked-prs branch August 19, 2026 16:22
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