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 a Cloud Build config to deploy website to Firebase. #5447

Merged
merged 12 commits into from
Jan 4, 2024

Conversation

khanhnwin
Copy link
Contributor

@khanhnwin khanhnwin commented Jan 4, 2024

As the title says. This is the Cloud Build configuration that we'll use to build dart.dev and publish to Firebase hosting.

Note: This current config currently publishes to a staging Flutter project to confirm everything works on main.

Once this PR lands and we've confirmed deploy works as expected,

cc @drewroengoogle @parlough

@dart-github-bot
Copy link
Collaborator

dart-github-bot commented Jan 4, 2024

Visit the preview URL for this PR (updated for commit 3f98199):

https://dart-dev--pr5447-cloud-build-deploy-xuxan9xb.web.app

args:
- '-c'
- |-
firebase deploy --project=flutter-website-staging-349021 --only=hosting
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this supposed to be to a dart project instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes! I'm going to open another PR to change that once this lands and we've confirmed it deploys properly. I don't want to remove the current deploy GitHub action until we confirm this deploy works & I don't want them to deploy on top of each other.

Copy link
Member

@parlough parlough left a comment

Choose a reason for hiding this comment

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

🫡

Looks good to me, thanks! Let's try it out 🚀

@khanhnwin khanhnwin merged commit 19c4f7d into main Jan 4, 2024
9 checks passed
@khanhnwin khanhnwin mentioned this pull request Jan 4, 2024
3 tasks
@parlough parlough deleted the cloud-build-deploy branch January 4, 2024 23:34
khanhnwin added a commit that referenced this pull request Jan 4, 2024
Addresses remaining AI's from #5447.

- [x] Swapped out deploy project to `dart-dev`
- [x] I've manually disabled the [deploy.yaml GitHub
action](https://github.com/dart-lang/site-www/blob/main/.github/workflows/deploy.yml)
workflow from the "Actions" menu.
- [x] Updated the Cloud Build trigger to go off on push to `main` branch
atsansone pushed a commit to atsansone/site-www that referenced this pull request Jan 26, 2024
As the title says. This is the Cloud Build configuration that we'll use
to build dart.dev and publish to Firebase hosting.

Note: This current config currently publishes to a staging Flutter
project to confirm everything works on main.

Once this PR lands and we've confirmed deploy works as expected, 
- [ ] I'll open another PR to switch to prod project `dart-dev`
- [ ] Decommission the [deploy.yaml GitHub
action](https://github.com/dart-lang/site-www/blob/main/.github/workflows/deploy.yml).
- [ ] Update the trigger branch to `main`

cc @drewroengoogle @parlough
atsansone pushed a commit to atsansone/site-www that referenced this pull request Jan 26, 2024
Addresses remaining AI's from dart-lang#5447.

- [x] Swapped out deploy project to `dart-dev`
- [x] I've manually disabled the [deploy.yaml GitHub
action](https://github.com/dart-lang/site-www/blob/main/.github/workflows/deploy.yml)
workflow from the "Actions" menu.
- [x] Updated the Cloud Build trigger to go off on push to `main` branch
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