Skip to content

fix(vercel): deploy landing/docs only on main when their files change#788

Merged
h4yfans merged 1 commit into
mainfrom
fix/vercel-deploy-conditions
Jul 18, 2026
Merged

fix(vercel): deploy landing/docs only on main when their files change#788
h4yfans merged 1 commit into
mainfrom
fix/vercel-deploy-conditions

Conversation

@h4yfans

@h4yfans h4yfans commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

What

  • ignoreCommand on landing + docs: skip preview (PR) deploys, and skip production deploys when that app's own files are unchanged.
  • installCommand with SKIP_ELECTRON_REBUILD=1: stop the workspace install failing on desktop's better-sqlite3 electron-rebuild.
  • Drop the "*" deploymentEnabled key — it is a literal branch name, not a wildcard, so it never disabled PR previews.

Why

Every merge to main redeployed landing/docs for unrelated desktop/sync changes, PR branches deployed failing previews, and every deploy since the Electron 43 bump (#765) errored on pnpm install.

@vercel

vercel Bot commented Jul 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
memry-docs Ignored Ignored Preview Jul 18, 2026 8:22am
memrynote-landing Ignored Ignored Preview Jul 18, 2026 8:22am

@github-actions github-actions Bot added bug Something isn't working documentation Improvements or additions to documentation labels Jul 18, 2026
- landing reads apps/landing/vercel.json; docs (memry-docs) reads the root
  vercel.json because its Root Directory is the repo root. Both now skip
  preview (PR) deploys, and skip production deploys when their own files are
  unchanged, via ignoreCommand.
- installCommand sets SKIP_ELECTRON_REBUILD=1 so the workspace install stops
  failing on desktop's better-sqlite3 electron-rebuild (broke at Electron 43).
- Drop the '*' deploymentEnabled key: a literal branch name, not a wildcard,
  so it never disabled PR previews.
@h4yfans
h4yfans force-pushed the fix/vercel-deploy-conditions branch from c7b394a to 3852e58 Compare July 18, 2026 08:21
@h4yfans
h4yfans marked this pull request as ready for review July 18, 2026 08:32
@h4yfans
h4yfans merged commit 0c204a2 into main Jul 18, 2026
19 checks passed
@codecov

codecov Bot commented Jul 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant