Skip to content

ci(website): Skip Renovate docs-site previews for manifest-only changes #4158

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ntucker merged 2 commits into from
Oct 4, 2026
Merged

ci(website): Skip Renovate docs-site previews for manifest-only changes #4158

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0b24a8a
ci(website): Skip docs-site previews that die in yarn prepare
cursoragent Oct 4, 2026
9085394
ci(website): Simplify to a Renovate manifest-only preview skip
cursoragent Oct 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .cursor/rules/ci-config.mdc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,3 +32,4 @@ alwaysApply: false

- Vercel's ignore step: exit 0 skips, anything else builds, so fail open. Never diff only `HEAD^`: a master merge or multi-commit push makes it wrong. Run `vercel-ignore.test.sh` after changes.
- `git.deploymentEnabled: false` doesn't stick (dashboard overrides it); skipped pushes still show as canceled deployments.
- Previews on `renovate/*` skip when the only site changes are website `package.json` or lockfiles; production is unchanged.
8 changes: 8 additions & 0 deletions website/scripts/vercel-ignore.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,14 @@ if [[ "${VERCEL_GIT_COMMIT_REF:-}" =~ ^(master|rest-hooks-site)$ || "${VERCEL_EN
build "no previous production deploy to compare"
fi

# Renovate previews skip when only website dependency manifests or lockfiles
# changed. Site or docs source changes still build.
if [[ "${VERCEL_GIT_COMMIT_REF:-}" == renovate/* ]]; then
for f in package.json yarn.lock package-lock.json pnpm-lock.yaml npm-shrinkwrap.json; do
SITE_PATHS+=(":(exclude,glob)website/**/$f")
done
fi

# Previews compare the branch's changes, not commits merged in from upstream.
# When the tip merges master, diff against the merged master commit: that
# counts the branch's site files and any conflict resolutions, not master's.
Expand Down
23 changes: 23 additions & 0 deletions website/scripts/vercel-ignore.test.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -180,4 +180,27 @@ expect build "production env site tip" other-branch "$(parent)" production
commit "prod pkg" packages/normalizr/src/index.ts
expect skip "production env package tip" other-branch "$(parent)" production

# --- Renovate previews ignore website dependency manifests and lockfiles ---
git -C "$repo" checkout -b renovate/docusaurus master >/dev/null 2>&1
commit "bump docusaurus" website/package.json
expect skip "renovate website package.json" renovate/docusaurus
expect build "renovate ref in production env" renovate/docusaurus "$(parent)" production
commit "bump lockfiles" website/yarn.lock website/examples/demo/package.json website/examples/demo/pnpm-lock.yaml
expect skip "renovate nested manifests and lockfiles since last deploy" renovate/docusaurus "$(parent)"
git -C "$repo" checkout master >/dev/null 2>&1
commit "master site for renovate" docs/core/api/Renovate.md
git -C "$repo" checkout renovate/docusaurus >/dev/null 2>&1
git -C "$repo" merge --no-edit master >/dev/null
expect skip "renovate merge of master" renovate/docusaurus
commit "renovate site source" website/src/pages/index.js
expect build "renovate with site source" renovate/docusaurus "$(parent)"

git -C "$repo" checkout -b deps master >/dev/null 2>&1
commit "manual bump" website/package.json
expect build "non-renovate website package.json" deps

git -C "$repo" checkout master >/dev/null 2>&1
commit "master bump" website/package.json website/yarn.lock
expect build "master website manifest" master "$(parent)"

echo "all vercel-ignore cases passed"
Loading