Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ci step to deploy firebase functions #2909

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sdowell
Copy link
Contributor

@sdowell sdowell commented Mar 17, 2022

The existing deployment step deploys the static hosted content on merge. This change is intended to additionally add a step which deploys the recently added firebase functions upon merge.

@sdowell sdowell requested review from mengqiy, mortent and yuwenma March 17, 2022 20:56
@sdowell sdowell force-pushed the kpt-dev-vanity-server branch from 9de71e3 to 4c8bb59 Compare March 17, 2022 21:27
@droot
Copy link
Contributor

droot commented Mar 22, 2022

@mengqiy Can you pl. take a look at this one ?

This change is intended to automatically deploy firebase functions in
addition to the already deployed hosted content on merge.
@sdowell sdowell force-pushed the kpt-dev-vanity-server branch from 0a3421f to 9cea2ef Compare September 7, 2022 20:13
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.

4 participants