Skip to content

Conversation

annajowang
Copy link
Contributor

@annajowang annajowang commented Sep 25, 2025

Sets the deploy context config for localBuild backends

  • In prepare, call the local build and set the build outputs in the config
  • In deploy, zip and upload the build directory

@annajowang annajowang changed the base branch from master to createArchive-subdir September 25, 2025 00:38
@annajowang annajowang changed the title Deploy local builds Set local build configs throughout the deploy steps Sep 25, 2025
@annajowang
Copy link
Contributor Author

/gemini review

@annajowang annajowang removed the request for review from Yuangwang September 25, 2025 15:54
@annajowang annajowang marked this pull request as ready for review September 25, 2025 15:54
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces support for local builds in App Hosting. The changes correctly modify the prepare and deploy steps to handle local build artifacts, including creating separate GCS buckets and passing build configurations. My review includes suggestions to improve type safety, make the code more robust against unexpected build outputs, and clarify logging messages for a better user experience.

Base automatically changed from createArchive-subdir to jojwang-localbuild September 25, 2025 16:23
@annajowang annajowang linked an issue Sep 25, 2025 that may be closed by this pull request
@annajowang annajowang merged commit c34bbef into jojwang-localbuild Sep 25, 2025
44 checks passed
@github-project-automation github-project-automation bot moved this from Approved [PR] to Done in [Cloud] Extensions + Functions Sep 25, 2025
@annajowang annajowang deleted the deploy-local-builds branch September 25, 2025 18:03
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.

Add support for apphosting local builds in firebase deploy
2 participants