Skip to content

docs: fix DEPLOYMENT_VERCEL.md vercel.json examples#190

Open
andrerfneves wants to merge 1 commit into
mainfrom
maintenance/fix-vercel-deploy-docs-20260617
Open

docs: fix DEPLOYMENT_VERCEL.md vercel.json examples#190
andrerfneves wants to merge 1 commit into
mainfrom
maintenance/fix-vercel-deploy-docs-20260617

Conversation

@andrerfneves

Copy link
Copy Markdown
Owner

Summary

Updates the vercel.json examples in DEPLOYMENT_VERCEL.md to match the project's actual configuration and prevent users from accidentally overwriting existing build settings.

Changes

Configuration section

  • The example showed unnecessary default fields (devCommand, installCommand) and was missing "framework": "vite" which tells Vercel to use Vite-optimized settings
  • Updated to reflect the actual vercel.json already in the project
  • Changed phrasing from "Create a vercel.json file" to "The project includes a vercel.json" since the file already exists

Troubleshooting section (404 errors)

  • The rewrites-only example would overwrite the project's existing buildCommand and outputDirectory config if copied verbatim
  • Changed to show the full vercel.json with rewrites merged in, preventing accidental configuration loss

Test Plan

  • No behavior change — documentation-only fix
  • All 52 existing tests pass
  • Markdown rendering verified locally

…onfig

The Configuration section showed a vercel.json example with
unnecessary default fields (devCommand, installCommand) and was
missing the framework field that tells Vercel to use Vite-optimized
settings. Updated to match the project's actual vercel.json.

The Troubleshooting rewrites example showed only a rewrites block,
which would overwrite the project's existing build configuration
if copied verbatim. Changed to show the full config with rewrites
merged in.
@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
lightning-decoder Ready Ready Preview, Comment Jun 17, 2026 10:04am

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