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

Custom GitHub Actions workflow to continuously deploy main to the Heroku staging app #9494

Merged
merged 12 commits into from
Nov 22, 2022

Conversation

tbrlpld
Copy link
Collaborator

@tbrlpld tbrlpld commented Oct 14, 2022

This PR enables a custom GitHub Actions workflow to deploy the main branch to the staging app on Heroku.

The custom workflow checks that the CI pipline has finished successfully, checks out the main branch and pushes it to Heroku via the traditional Git push deployment.

The docs are updated to reflect this change and contain information on which credentials need to be set up and how to get them.

I have tested that the Heroku access works with a simple git fetch heroku main. This is the workflow run of the test: https://github.com/mozilla/foundation.mozilla.org/actions/runs/3492453124/jobs/5846247852

Related PRs/issues: #9215

@tbrlpld tbrlpld force-pushed the 9215-continous-deployment-github-action branch 2 times, most recently from 66dc0b9 to 50ae3b6 Compare November 17, 2022 22:55
@tbrlpld tbrlpld marked this pull request as ready for review November 17, 2022 23:08
@tbrlpld tbrlpld changed the title Experimental CD workflow to push to Heroku Custom GitHub Actions workflow to continuously deploy main to the Heroku staging app Nov 17, 2022
@mtdenton
Copy link
Contributor

This looks good to me, but leaving this for someone else to approve so that I'm not the only one checking this out. Thanks @tbrlpld

Copy link
Collaborator

@danielfmiranda danielfmiranda left a comment

Choose a reason for hiding this comment

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

Hi @tbrlpld,

This PR looks good to me. Thanks again for taking this work on and updating the documentation as well. This will be very helpful in the future.

Approved!

Copy link
Contributor

@fessehaye fessehaye left a comment

Choose a reason for hiding this comment

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

I had one small question but its not a blocker!

Copy link
Collaborator

@mmmavis mmmavis left a comment

Choose a reason for hiding this comment

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

Thanks for taking this on @tbrlpld ! I don't have a way to test this out but code and documentation both look great to me! 👍🏻

@tbrlpld tbrlpld force-pushed the 9215-continous-deployment-github-action branch from 22b557c to a867749 Compare November 22, 2022 21:57
@tbrlpld tbrlpld merged commit a1dc2a3 into main Nov 22, 2022
@tbrlpld tbrlpld deleted the 9215-continous-deployment-github-action branch November 22, 2022 22:04
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.

5 participants