chore: enable auto-format on PRs targeting production#31208
chore: enable auto-format on PRs targeting production#31208MohamedH1998 wants to merge 1 commit into
Conversation
Review✅ No style-guide issues found in commit CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
There was a problem hiding this comment.
Clean infrastructure change — no issues.
What it does: Changes the branch filter from to , enabling the auto-format bot to run on real PRs.
Checks passed:
- Branch name () matches the repo's primary branch.
- Companion workflow () correctly triggers via and needs no changes.
- Bot-loop guard () is in place.
- Patch validation, path allowlisting, and stale-head checks in the apply workflow are unchanged and effective.
- The CI formatting safety net in remains as described.
|
Auto-review note: the backtick-quoted workflow filenames above got swallowed by shell interpolation. The review is an approval with no blocking issues. The diff correctly flips → in . |
|
Review complete for PR #31208 — approved with no blocking issues. Summary of changesChanges a single line in Labels applied
Issues flaggedNone. The change is minimal and correct. Verification checked
|
|
Preview URL: https://fa746799.preview.developers.cloudflare.com |
Summary
Flips the auto-format build workflow's
branches:filter from[auto-format-canary]to[production], so the bot now runs on real PRs.Verified end-to-end against canary PRs: prettier produces a diff for changed files, the patch is validated against the path allowlist, and
cloudflare-docs-bot[bot]pushes astyle: formatcommit back to the PR branch.Effect
The auto-format bot will now fire on every PR opened against
productionthat touches a prettier-scoped file (.js .jsx .ts .tsx .mjs .css .json .yaml .yml .md .astro). When prettier produces a diff,cloudflare-docs-bot[bot]pushes astyle: formatcommit back to the PR branch (same-repo or fork), CI re-runs against the formatted code.The CI safety net (
Check formatting (changed files)inci.yml) is retained — it'll be removed in a follow-up after a soak period.Verification
The workflow has been tested against PR targeting
auto-format-canarywithout any issues:Screenshots (optional)
Documentation checklist