ci: gate PR preview builds#603
Conversation
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3ac3c4c267
ℹ️ 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".
| - closed | ||
| paths: | ||
| - "website/**" | ||
| - ".github/actions/build-docs/action.yml" |
There was a problem hiding this comment.
Include setup action changes in preview triggers
When a PR changes only .github/actions/setup-node-pnpm/action.yml, this path filter skips the preview workflow even though the docs build action invoked below delegates its toolchain setup to that composite action (.github/actions/build-docs/action.yml:12-14). That means changes to the Node/pnpm install path can merge without exercising the PR preview build they affect; add the setup action path (or the whole .github/actions/** subtree) to this trigger list.
Useful? React with 👍 / 👎.
926a8b5 to
484f1fc
Compare
Summary
Only build and deploy PR previews when docs-preview inputs change. The preview workflow now runs for opened/reopened/synchronized PRs that touch website/docs build inputs, while closed PR cleanup is handled by a separate closed-only workflow without path filters.
This keeps preview cleanup reliable and avoids the docs install/build/deploy path for code-only PRs.
Touched-file count: 2. Scope stayed within PR preview workflow plumbing.
Validation
Validated workflow edits with
git diff --checkand local YAML parsing for both preview workflows. This PR itself is expected to run the preview workflow because it edits.github/workflows/pr-preview.yml; the skip behavior applies to subsequent code-only PRs after merge.