vercel icon indicating copy to clipboard operation
vercel copied to clipboard

[gatsby] Add `@vercel/gatsby` Builder

Open lydiahallie opened this issue 1 year ago • 3 comments

This PR adds support for the following Gatsby 4, including SSR, DSG and API routes.

⚠️ Note: Redirects have to be added manually in the Vercel config file.

lydiahallie avatar Jul 27 '22 06:07 lydiahallie

The yarn.lock file appeared to have dupe entries for ncc. Maybe this was the result of a merge? It was causing this error:

ERROR  yarn.lock: could not unmarshal lockfile: yaml: unmarshal errors:
  line 3222: mapping key "@vercel/[email protected]" already defined at line 3191

I pulled this down, ran yarn install, and the updated yarn.lock appears to be OK. I pushed that commit up. If something else was wrong, I can revert this change.

EndangeredMassa avatar Sep 13 '22 20:09 EndangeredMassa

The tests are now passing here, but I think it's a false positive because vc build isn't actually using this Builder yet since it's not listed in the framework's useRuntime prop.

TooTallNate avatar Sep 14 '22 07:09 TooTallNate

It looks like we still need prepareCache() implemented in this Builder as well. The current configuration (when going through @vercel/static-build) is {.cache,public}/**.

TooTallNate avatar Sep 19 '22 18:09 TooTallNate

Thanks for all your help here! I know this is a large, gnarly addition 😄

leerob avatar Sep 24 '22 15:09 leerob

hi @lydiahallie - how can I test this on my build? Any idea when this will be merged?

rwitchell avatar Oct 23 '22 05:10 rwitchell

Deployment failed with the following error:

Creating the Deployment Timed Out.

vercel[bot] avatar Jan 04 '23 19:01 vercel[bot]

closing in favor of @vercel/gatsby-plugin-vercel-builder

Ethan-Arrowood avatar Jan 19 '23 23:01 Ethan-Arrowood