Skip to content

Stop Vercel from building the main line - #122

Closed
ledwards wants to merge 1 commit into
mainfrom
vercel-disable-main-line
Closed

Stop Vercel from building the main line#122
ledwards wants to merge 1 commit into
mainfrom
vercel-disable-main-line

Conversation

@ledwards

@ledwards ledwards commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Two files, no behavior change to the site. Split out of #121 so it can land on its own — it's unrelated to the static-mirror work and fixes the red check on main and on every open PR.

The problem

The Vercel project is connected at the repo level, so it attempted a build on every branch. Only ai-incarnations carries a vercel.json; on this line Vercel had nothing to build, so every attempt failed and posted a red Vercel check — on main itself and on PRs that have nothing to do with Vercel.

It wasn't a stale integration. It was a real failure of a deployment that was never supposed to happen here.

The fix

git.deploymentEnabled: falsedocumented as turning off automatic deployments for branches carrying the file.

ai-incarnations is unaffected. It carries its own vercel.json without that key and its builds are green today. The README says so, and says not to reconcile the two files, because the difference is the point.

Verified

  • Same change on Generate the static mirror at deploy time instead of committing it #121 already made the failing Vercel check disappear there — test passes, no red check
  • ai-incarnations still reports Vercel: success
  • main has no branch protection and no required status checks, so nothing depended on the Vercel context existing. I checked before making a check vanish rather than turn green
  • root.vc is Netlify: server: Netlify and x-nf-request-id on the live response, apex A at 75.2.60.5, www 301s to apex

Note for whoever merges

vercel.json here is byte-identical to the copy on #121, so the two merge cleanly in either order. The README section lives only here to avoid a conflict.

The remaining Vercel Agent Review check reports skipping — insufficient Credit. That's a billing state on the Vercel account, unrelated, and it reports neutral rather than failing.

🤖 Generated with Claude Code

ledwards added a commit that referenced this pull request Aug 3, 2026
The Vercel project is connected at the repo level, so it attempted a
build on every branch. Only ai-incarnations carries a vercel.json, so
builds on this line had nothing to read and failed — posting a red check
on this PR, which has nothing to do with Vercel.

root.vc is served by Netlify. Set git.deploymentEnabled: false so
branches carrying this file don't trigger a Vercel deployment.

Byte-identical to the copy on #122, which lands the same file on main
along with the README rationale. The two merge cleanly in either order.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The Vercel project is connected at the repo level, so it attempted a
build on every branch. Only ai-incarnations carries a vercel.json, so
builds on this line had nothing to read and failed — posting a red check
on PRs unrelated to Vercel, and on main itself.

root.vc is served by Netlify. Set git.deploymentEnabled: false so
branches carrying this file don't trigger a Vercel deployment.

ai-incarnations has its own vercel.json without that key and keeps
deploying successfully, so the parked experiment is unaffected. The
README records why both files exist and how to resolve the conflict if
that branch is ever merged down.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@ledwards
ledwards force-pushed the vercel-disable-main-line branch from f17fb81 to fb22f10 Compare August 3, 2026 23:47
ledwards added a commit that referenced this pull request Aug 4, 2026
Records that Netlify serves root.vc from dist/, that the Vercel project
belongs to the parked ai-incarnations branch, and why the two vercel.json
files deliberately differ — so nobody reconciles them and breaks that
branch's deploys.

Folded in from #122 so this line carries the whole change.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@ledwards

ledwards commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #120, which now carries a byte-identical vercel.json plus the README hosting section folded in from here.

Consolidating so there is one PR to review and merge rather than two touching the same file. Nothing is lost — the Vercel fix reaches main when #120 lands.

@ledwards ledwards closed this Aug 4, 2026
@ledwards
ledwards deleted the vercel-disable-main-line branch August 4, 2026 01:28
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.

1 participant