Skip to content

chore: remove Netlify and Vercel config after the move to Cloudflare - #7060

Merged
jennifer-shehane merged 1 commit into
mainfrom
claude/remove-netlify
Nov 6, 2026
Merged

jennifer-shehane merged 1 commit into
mainfrom
claude/remove-netlify

Conversation

@jennifer-shehane

@jennifer-shehane jennifer-shehane commented Nov 6, 2026 •

Copy link
Copy Markdown
Member

Note

Rebased onto main after cypress-io/cypress-documentation#7059 merged. That PR's /app/cloud/* redirect, which the new guidance in AGENTS_REFERENCE.md refers to, is in place in static/_redirects. After this merges, turn off the Netlify site (steps at the end).

Summary

Removes the Netlify and Vercel hosting config now that docs.cypress.io is served by the cypress-documentation Cloudflare Worker. static/_redirects becomes the only redirect list. The contributor and agent guides, README, and code comments now describe the Cloudflare setup.

Why

Production moved to Cloudflare in cypress-io/cypress-documentation#7058. Since then, netlify.toml and vercel.json configure hosts that no longer serve the site. While they stay in the repo, contributors are told to keep two redirect lists in sync, and the docs point outside contributors to Netlify preview links that will stop existing.

We also decided that PRs from forks won't get preview links. The Cloudflare preview needs repository secrets that GitHub withholds from forks, and that's an acceptable tradeoff. CONTRIBUTING now says so, and tells outside contributors how to check their changes locally instead.

What changed

  • Deleted netlify.toml. All of its redirect rules were already in static/_redirects, in the same order. Its sitemap plugin isn't needed: Docusaurus builds sitemap.xml itself, with the same 338 pages (without Netlify's stray /404 entry).
  • Deleted vercel.json. Its 6 redirects were already in static/_redirects. Removed .vercel from .gitignore.
  • Redirect guidance (AGENTS.md, AGENTS_REFERENCE.md, .cursor/BUGBOT.md): points only at static/_redirects, and adds two placement rules Cloudflare needs:
  • CONTRIBUTING.md: replaced the Netlify preview instructions and screenshot with the cloudflare/preview status and its pr-<number> URL, plus the fork note.
  • README.md: removed the Netlify status badge.
  • Comments in ci.yml, tag-docs-release.yml, scripts/report-tagging-failure.mjs, and wrangler.jsonc no longer compare things to Netlify.

The Netlify mentions in docs/app/run-tests/continuous-integration/overview.mdx, the changelog, and docs/cloud/guides/webhooks.mdx stay. They're about Cypress users' own Netlify setups, not about where these docs are hosted.

Notes for reviewers

  • Verified locally:
    • npm run lint:fix, npm run build, and npm run typecheck pass.
    • npx wrangler deploy --dry-run reads all 9,336 files.
    • npm run serve answers on port 3000, as the new CONTRIBUTING text says.
    • A repo-wide search finds no remaining references to Netlify or Vercel as our host. I reran it after the rebase.
  • The rebase: its only conflict was fix: send old /guides/cloud/ URLs to /cloud/ on Cloudflare #7059's edit to netlify.toml, which this PR deletes. I resolved it by keeping the deletion.
  • Outside the repo, after merging: stop builds for the cypress-docs Netlify site, remove docs.cypress.io from it, and remove this repository from the Netlify GitHub App, so its preview and rule checks stop appearing on PRs. None of these are required checks in branch protection.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WUv8r4AAgMMw1DSbt1H69h


Note

High Risk
Drops two legacy redirect layers; any mismatch with static/_redirects would break inbound URLs in production, so reviewers should verify parity before merge.

Overview
Cloudflare is now the documented deployment path. CONTRIBUTING.md, README.md, Bugbot rules, and both AGENTS docs only mention the Cloudflare preview (cloudflare/preview status plus the pr-<number>-cypress-documentation.cypress-io.workers.dev URL) and retire Netlify/Vercel instructions.

Hosting config cleanup mirrors that shift. netlify.toml, vercel.json, and .vercel in .gitignore are removed, while redirect guidance narrows to static/_redirects with extra Cloudflare wildcard cautions and comments in wrangler.jsonc, ci.yml, tag-docs-release.yml, and scripts/report-tagging-failure.mjs describing the Worker-based flow.

Risk note: confirm static/_redirects already contained every rule from the deleted hosts so inbound links keep working; if anything drifts, the traffic loss would show up in production only.

Reviewed by Cursor Bugbot for commit 4a02a8a. Bugbot is set up for automated code reviews on this repo. Configure here.

docs.cypress.io is served by the cypress-documentation Cloudflare
Worker, so netlify.toml and vercel.json no longer configure anything.
static/_redirects becomes the only redirect list: netlify.toml carried
the same rules in the same order, and vercel.json's six were already in
it.

The contributor and agent guides now point at static/_redirects alone,
and add the two placement rules Cloudflare needs: exact paths above the
wildcards, and no new wildcard that overlaps another one. CONTRIBUTING
describes the Cloudflare preview, and says that PRs from forks don't get
one, so outside contributors know to check their changes locally.
Comments that compared things to Netlify now say how they work today.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WUv8r4AAgMMw1DSbt1H69h
@cypress

cypress Bot commented Nov 6, 2026

Copy link
Copy Markdown

cypress-documentation    Run #2792

Run Properties:  status check passed Passed #2792  •  git commit 0d2df5be47 ℹ️: Merge 4a02a8aef3e72241bdb87c35f55ff81755f376cd into de18c7c344ba9896304d06fc07fb...
Project cypress-documentation
Branch Review claude/remove-netlify
Run status status check passed Passed #2792
Run duration 02m 13s
Commit git commit 0d2df5be47 ℹ️: Merge 4a02a8aef3e72241bdb87c35f55ff81755f376cd into de18c7c344ba9896304d06fc07fb...
Committer Jennifer Shehane
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 427
View all changes introduced in this branch ↗︎
UI Coverage  32.85%
  Untested elements 738  
  Tested elements 25  
Accessibility  98.93%
  Failed rules  2 critical   5 serious   4 moderate   1 minor
  Failed elements 74  

@jennifer-shehane
jennifer-shehane merged commit fc274e3 into main Nov 6, 2026
25 checks passed
@jennifer-shehane
jennifer-shehane deleted the claude/remove-netlify branch November 6, 2026 19:06
@cypress

cypress Bot commented Nov 6, 2026

Copy link
Copy Markdown

cypress-documentation    Run #2793

Run Properties:  status check passed Passed #2793  •  git commit fc274e3d59: chore: remove Netlify and Vercel config after the move to Cloudflare (#7060)
Project cypress-documentation
Branch Review main
Run status status check passed Passed #2793
Run duration 02m 16s
Commit git commit fc274e3d59: chore: remove Netlify and Vercel config after the move to Cloudflare (#7060)
Committer Jennifer Shehane
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 427
View all changes introduced in this branch ↗︎
UI Coverage  32.91%
  Untested elements 737  
  Tested elements 26  
Accessibility  98.9%
  Failed rules  2 critical   6 serious   4 moderate   1 minor
  Failed elements 153  

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