Closed mythmon closed 1 month ago
If this LGTY it LGTM. 👍
Looks like you’ll have a bit of a chore merging against #1295 though. 😬
I tried to merge this with git merge
, and like you predicted it was looking pretty difficult to work with. Instead I reverted deploy.ts
back to it's state on main, and manually applied all the changes from this branch to the right logical places. Much easier.
Thanks for the updates, and especially the doc changes.
Add a --deploy-config option to deploy, to allow specifying alternate paths for
deploy.json
. This can help support our examples, as well as users that want to maintain multiple deploys, such as stage and prod projects.