Noticed a minor extra complexity with how the vercel deployment works. instead of uploading dist/now and moving all the gh-pages into that folder, I suggest we just upload dist.

that sounds good to me. this PR basically removes the "now" term from our file structure, which is a great addition to #12901. (i was thinking we do s/now/vercel/ but i like this proposal even more)

can you adjust the two lines in dogfood-lhci.sh that also work with /now/ ?

Noticed a minor extra complexity with how the vercel deployment works. instead of uploading dist/now and moving all the gh-pages into that folder, I suggest we just upload dist.

that sounds good to me. this PR basically removes the "now" term from our file structure, which is a great addition to #12901. (i was thinking we do s/now/vercel/ but i like this proposal even more)

FWIW https://vercel.com/guides/upgrade-to-zero-configuration#frameworks-with-zero-configuration makes it seem like we could drop the now.json too if we're willing to && mv dist public (edit: in yarn vercel-build, I mean, not everywhere :)

It does seem like a yarn build-all could sneak into this npm script chain at some point and we'll be uploading MiBs of javascript from dist/ with no one noticing, but presumably that would still be well within any deployment limits? :)

can you adjust the two lines in dogfood-lhci.sh that also work with /now/ ?

should something have failed here since ./dist/now/english/ didn't exist?

can you adjust the two lines in dogfood-lhci.sh that also work with /now/ ?

should something have failed here since ./dist/now/english/ didn't exist?

heh probably but this happened:

(and probably happens on all runs all the time. ruh roh)

FWIW vercel.com/guides/upgrade-to-zero-configuration#frameworks-with-zero-configuration makes it seem like we could drop the now.json too if we're willing to && mv dist public (edit: in yarn vercel-build, I mean, not everywhere :)

i tried this on another project recently.. unfortch the config file is still necessary if you don't want vercel to comment after every deployment. (the "github": {"silent": true}, bit)