Skip to content

fix(cli): add .vercel to .gitignore when using astro add vercel#15185

Merged
Princesseuh merged 1 commit intowithastro:mainfrom
EricGrill:fix/vercel-gitignore
Jan 22, 2026
Merged

fix(cli): add .vercel to .gitignore when using astro add vercel#15185
Princesseuh merged 1 commit intowithastro:mainfrom
EricGrill:fix/vercel-gitignore

Conversation

@EricGrill
Copy link
Contributor

Summary

When running astro add vercel, the Vercel adapter creates build artifacts in .vercel/output/. This directory should be gitignored to avoid committing build artifacts.

This PR adds automatic .gitignore handling for the Vercel adapter, following the same pattern already used for Cloudflare, Tailwind, Svelte, and other integrations:

  • If .gitignore exists but doesn't contain .vercel, prompts to add it
  • If .gitignore doesn't exist, prompts to create it with .vercel
  • If .vercel is already in .gitignore, skips silently

Closes #15058

Test plan

  1. Run astro add vercel in a project without .gitignore - should prompt to create one
  2. Run astro add vercel in a project with .gitignore without .vercel - should prompt to add it
  3. Run astro add vercel in a project with .gitignore containing .vercel - should skip silently

🤖 Generated with Claude Code

The Vercel adapter outputs build artifacts to .vercel/output which should be gitignored. This change automatically adds .vercel to .gitignore when running `astro add vercel`, following the same pattern used for Cloudflare, Tailwind, and other integrations.

Closes withastro#15058

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@changeset-bot
Copy link

changeset-bot bot commented Jan 13, 2026

🦋 Changeset detected

Latest commit: 0e58495

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Jan 13, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 13, 2026

Merging this PR will not alter performance

✅ 9 untouched benchmarks


Comparing EricGrill:fix/vercel-gitignore (0e58495) with main (37ab65a)1

Open in CodSpeed

Footnotes

  1. No successful run was found on main (7f449b7) during the generation of this report, so 37ab65a was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

Copy link
Member

@Princesseuh Princesseuh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thank you!

@Princesseuh Princesseuh merged commit edabeaa into withastro:main Jan 22, 2026
26 checks passed
@astrobot-houston astrobot-houston mentioned this pull request Jan 22, 2026
florian-lefebvre added a commit that referenced this pull request Jan 30, 2026
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Emanuele Stoppa <[email protected]>
Co-authored-by: Matt Kane <[email protected]>
Co-authored-by:  Matthew Phillips <[email protected]>
Co-authored-by: Matthew Phillips <[email protected]>
Co-authored-by: Florian Lefebvre <[email protected]>
Co-authored-by: Erika <[email protected]>
Co-authored-by: Sarah Rainsberger <[email protected]>
Co-authored-by: Armand Philippot <[email protected]>
Co-authored-by: matthewp <[email protected]>
Co-authored-by: florian-lefebvre <[email protected]>
Co-authored-by: ematipico <[email protected]>
Co-authored-by: Luiz Ferraz <[email protected]>
Co-authored-by: HiDeoo <[email protected]>
Co-authored-by: Princesseuh <[email protected]>
Co-authored-by: Chris Swithinbank <[email protected]>
Co-authored-by: Claude Opus 4.5 <[email protected]>
Co-authored-by: Antony Faris <[email protected]>
Co-authored-by: Erika <[email protected]>
Co-authored-by: Houston (Bot) <[email protected]>
Co-authored-by: Florian Lefebvre <[email protected]>
Co-authored-by: Volpeon <[email protected]>
Co-authored-by: Martin Trapp <[email protected]>
Co-authored-by: fkatsuhiro <[email protected]>
Co-authored-by: Oliver Speir <[email protected]>
Co-authored-by: Andreas Deininger <[email protected]>
Co-authored-by: Roman <[email protected]>
Co-authored-by: fabon <[email protected]>
Co-authored-by: Raanelom <[email protected]>
Co-authored-by: Rahul Dogra <[email protected]>
Co-authored-by: James Garbutt <[email protected]>
Co-authored-by: Pegasus <[email protected]>
Co-authored-by: Cameron Smith <[email protected]>
Co-authored-by: Martin Trapp <[email protected]>
Co-authored-by: Rafael ヤスヒデ 須藤 <[email protected]>
Co-authored-by: Matthew Phillips <[email protected]>
Co-authored-by: Julien Cayzac <[email protected]>
Co-authored-by: Drew Powers <[email protected]>
Co-authored-by: Emanuele Stoppa <[email protected]>
Co-authored-by: MkDev11 <[email protected]>
Co-authored-by: andy <[email protected]>
Co-authored-by: Luky Setiawan <[email protected]>
Co-authored-by: btea <[email protected]>
Co-authored-by: cid <[email protected]>
Co-authored-by: Simen Sagholen Førrisdal <[email protected]>
Co-authored-by: Alex Launi <[email protected]>
Co-authored-by: Kedar Vartak <[email protected]>
Co-authored-by: yy <[email protected]>
Co-authored-by: Matheus Baroni <[email protected]>
Co-authored-by: Kevin Brown <[email protected]>
Co-authored-by: Eric Grill <[email protected]>
Co-authored-by: Cameron Pak <[email protected]>
Co-authored-by: Tony Narlock <[email protected]>
Co-authored-by: John Mortlock <[email protected]>
Co-authored-by: Patrick Arlt <[email protected]>
Co-authored-by: Mark Ignacio <[email protected]>
fix(ci): Reinstall deps after having published VS Code (#14996)
fix(svelte): allow client directives (#15004)
fix(assets): Fixes missing format option for svgs in the passthrough service (#14987)
fix(deps): update all non-major dependencies (#15020)
fix(content-layer): Try a smarter solution to normalize bare image paths in JSON (#15028)
fix(astro): assets vite build log (#15034)
resolved (#15033)
fix: Remote images: Prevent internal caching from interfering with Astro's cache (#14954)
fix(deps): update astro adapters (#15084)
fix(deps): update all non-major dependencies (#15072)
fix(deps): update astro client runtimes (#15085)
fix: move ts-plugin node_modules to dist (#15083)
fix: components imports paths (#15107)
fix(assets): Use Vite's isFileLoadingAllowed to check if a file can be loaded (#15052)
fix(vscode): Correctly handle TypeScript blocks ending with types (#15109)
fix(svelte): improve Svelte children prop type checking (#15070)
fix Firefox e2e tests for playwright 1.57 (#15113)
fix(deps): update astro dependencies (#15103)
fix: lint vt test (#15114)
fix(deps): update language tools (#15104)
resolver abstraction (#15111)
fix(vue): add HTML attributes to generated TypeScript types (#15016)
Fixes #14686
fix(vscode): Don't update @types/vscode automatically (#15131)
fix: adjust page warning to only show up in more relevant times (#15127)
fix(dev): preserve query params when base path is stripped (#15124)
fix(assets): hoist ?? inside JSON.stringify in virtual module codegen (#15140)
fix(toolbar): skip image audit for framework components (#15149)
Fixes #15048
fix: Accept setCookie from both context and headers (#15152)
fix(ci): Move ts-plugin-bundle to node_modules after recreating node_modules (#15156)
fix: scripts not rendering with unused Fragment slots (#13847) (#15147)
Fix hydration for framework components in MDX slots (#15150)
fix(deps): update astro adapters (#15173)
fix: fix image 500 error when moving dist directory in standalone Node (#15169)
fix: Allow node: prefix for Node builtins for Vercel middleware (#14863)
fix: Allow node: prefix for Node builtins for Vercel middleware (#14839)"
fix(node): hash URL stripping (#15196)
fix(core): add defensive validation for mod.page in App.render (#15148)
fix(css): rewrite absolute URLs with base path in dev mode (#14622)
Closes #14585
fix(docs): replace outdated Astro docs links (#15199)
fix(lint): Move ESLint comment (#15216)
fix: update devalue to the latest (#15222)
fix(css): prevent CSS double-bundling (#14991) (#15017)
Fixes #14991
fix: assets referenced via js chunks are not flagged as orphaned (#14607)
fix(build): Prevent duplicate CSS for hydrated client components" (#14612)"
Fixes #14252
fix(deps): update astro client runtimes (#15240)
fix(deps): update astro adapters (#15243)
fix(deps): update astro dependencies (#15241)
fix(deps): update dependency prettier to ^3.8.0 (#15244)
fix(deps): update all non-major dependencies (#15242)
Fix greedy regex in error message markdown rendering (#15230)
Fixes #15068
fixes: #15252
fix: prevent font copying when stopping dev server with q+enter (#15178)
fix: renovate config typo (#15256)
Fixes #15251
fix: typo in comment (#15232)
fix(deps): update dependency svelte to v5.46.4 [security] (#15220)
fix: add favicon.ico fallbacks to all examples (#15262)
fix(cli): add .vercel to .gitignore when using astro add vercel (#15185)
Closes #15058
fix(fonts): font providers as class instances (#15286)
Fix: Remove await from getActionResult JSDoc example (#15304)
fix(errors): Only style valid URLs in the error overlay (#15324)
fix(fonts): remove getFontBuffer() (#15334)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Astro Vercel integration missing .gitignore entry

2 participants