Skip to content

ci: skip docs CI for flue-only PRs, add Flue CI workflow - #32178

Merged
mvvmm merged 5 commits into
productionfrom
flue/ci-path-filter
Jul 20, 2026
Merged

ci: skip docs CI for flue-only PRs, add Flue CI workflow#32178
mvvmm merged 5 commits into
productionfrom
flue/ci-path-filter

Conversation

@mvvmm

@mvvmm mvvmm commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a paths-ignore: [".flue/**"] filter to ci.yml and a new flue-ci.yml workflow so that PRs touching only .flue/ skip the full docs build (Astro, link validation, deploy preview) and run a lightweight Flue-specific check instead.

ci.yml — adds paths-ignore: [".flue/**"] to the trigger. The workflow is not started at all when every changed file is under .flue/.

flue-ci.yml — new workflow triggered on paths: [".flue/**"] with three jobs that mirror the required status check names:

Job What it does
Pre Build pnpm run typecheck in .flue/
Build pnpm run build in .flue/ — compiles the Worker via flue build
Post Build No-op placeholder for future unit tests / evals

Since branch protection matches required checks by job name, a flue-only PR gets "Pre Build", "Build", and "Post Build" checks from flue-ci.yml instead of ci.yml, so merging is not blocked.

@github-actions

Copy link
Copy Markdown
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/.github/ @cloudflare/content-engineering, @kodster28, @mvvmm, @colbywhite, @ahaywood, @MohamedH1998

Comment thread .github/workflows/flue-ci.yml Fixed
Comment thread .github/workflows/flue-ci.yml Fixed
Comment thread .github/workflows/flue-ci.yml Fixed
mvvmm and others added 3 commits July 20, 2026 13:37
…ntain permissions'

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@mvvmm
mvvmm marked this pull request as ready for review July 20, 2026 18:54
@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Review

✅ No outstanding issues in commit a2ff0ad.

Code Review

This code review is in beta and may not always be helpful — use your judgment.

No code review issues found.

Conventions

No convention issues found.

Style Guide Review

No style-guide issues found.


Acknowledged by author (1)
Review File Issue Note
Code .github/workflows/flue-ci.yml line 69 Dead placeholder job The author says the post-build job is an intentional required-check stub needed so the Post Build status check is reported for flue-only pull requests.
Commands

Only codeowners can run commands. Post a comment with the command to trigger it.

Command Description
/review Runs a review now. Incremental if a prior review exists, full if not.
/full-review Re-reviews the entire PR diff from scratch, ignoring incremental history. Useful after a rebase, when you want a fresh review, or if the bot gets out of sync and reports issues that no longer exist.
/ignore-review-limit Permanently lifts the 2-review automatic limit for this PR. Future pushes will trigger reviews as normal.
/disable-auto-review Stops automatic reviews from triggering on future pushes to this PR. Codeowners can still run /review or /full-review manually.
/rebase Rebases the PR branch against production. Stops if there are conflicts and reports which files conflict.
/rebaseWithConflicts Rebases against production and attempts to resolve conflicts automatically using AI. Stops with an explanation if confidence is not high enough.

@mvvmm

mvvmm commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

CR-5e78317fda17 — this is addressed by the new flue-ci.yml workflow in this same PR. It has jobs named Pre Build, Build, and Post Build that run when .flue/** files change, satisfying any required status checks by name. The bot reviewed the two files independently rather than as a pair.

CR-bcde13235f0f — the post-build job is an intentional required-check stub, not a dead placeholder. Without it, the Post Build required status check is never reported for flue-only PRs, blocking merges. It will be replaced with real checks (unit tests, evals) when those are added.

@github-actions

Copy link
Copy Markdown
Contributor

@github-actions

Copy link
Copy Markdown
Contributor

@mvvmm
mvvmm merged commit 18977ea into production Jul 20, 2026
19 checks passed
@mvvmm
mvvmm deleted the flue/ci-path-filter branch July 20, 2026 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants