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

feat: add deploy workflow #543

Merged
merged 1 commit into from
Jan 13, 2025
Merged

feat: add deploy workflow #543

merged 1 commit into from
Jan 13, 2025

Conversation

gminn
Copy link
Member

@gminn gminn commented Jan 13, 2025

Summary

Currently, we only deploy the site on pushes to master. This works fine expect in the cases
where we might was to schedule a post with a future publish date. This is scheduled for 10am ET every weekday.

Test Plan

  • Manually triggered the workflow with
    gh workflow run deploy.yml --ref gminn/add-daily-deployment,
    confirmed it ran successfully (result)

  • Confirmed site was deployed properly 🎉

    CleanShot 2025-01-12 at 21 58 41@2x

Copy link

cloudflare-workers-and-pages bot commented Jan 13, 2025

Deploying interrupt with  Cloudflare Pages  Cloudflare Pages

Latest commit: ae7e9e2
Status: ✅  Deploy successful!
Preview URL: https://4bf70e3e.interrupt.pages.dev
Branch Preview URL: https://gminn-add-daily-deployment.interrupt.pages.dev

View logs

Copy link
Member Author

gminn commented Jan 13, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@gminn gminn force-pushed the gminn/add-daily-deployment branch from fbd5bea to 415b218 Compare January 13, 2025 02:32
 ### Summary

 Scheduled for 10am ET every weekday.

 ### Test Plan

 - Manually trigger the workflow to ensure it runs successfully
@gminn gminn force-pushed the gminn/add-daily-deployment branch from 415b218 to ae7e9e2 Compare January 13, 2025 02:34
@gminn gminn marked this pull request as ready for review January 13, 2025 02:59
@gminn gminn requested a review from a team as a code owner January 13, 2025 02:59
@gminn gminn requested a review from tyhoff January 13, 2025 03:00
Copy link
Member

@tyhoff tyhoff left a comment

Choose a reason for hiding this comment

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

Looks great!

Copy link
Member Author

gminn commented Jan 13, 2025

Merge activity

  • Jan 13, 1:25 PM EST: A user started a stack merge that includes this pull request via Graphite.
  • Jan 13, 1:26 PM EST: A user merged this pull request with Graphite.

@gminn gminn merged commit d4ace88 into master Jan 13, 2025
3 checks passed
@gminn gminn deleted the gminn/add-daily-deployment branch January 13, 2025 18:26
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.

2 participants