Skip to content

Commit

Permalink
Run GitHub Actions full time, disable CircleCI (#75)
Browse files Browse the repository at this point in the history
We'll check back in to make sure this is working well in a week or so, and then delete the CircleCI config entirely.
  • Loading branch information
Mr0grog authored Aug 29, 2024
1 parent c22f198 commit 5f59a8c
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 17 deletions.
31 changes: 18 additions & 13 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -61,16 +61,21 @@ workflows:
- upload-zoom-recordings:
delete_after_upload: false
dry_run: true
youtube-upload:
jobs:
- upload-zoom-recordings:
delete_after_upload: true
triggers:
- schedule:
# Every hour in the afternoon at :10 on weekdays
# (This is only in the afternoon so we can compare how this is doing
# vs. how GitHub Actions is doing and then maybe turn off Circle.)
cron: "10 13-23 * * 1-5"
filters:
branches:
only: main

# FIXME: This script is temporarily disabled in CircleCI. Once we are
# confident it's doing the job reliably on GitHub, delete this whole
# CircleCI config file.
#
# youtube-upload:
# jobs:
# - upload-zoom-recordings:
# delete_after_upload: true
# triggers:
# - schedule:
# # Every hour in the afternoon at :10 on weekdays
# # (This is only in the afternoon so we can compare how this is doing
# # vs. how GitHub Actions is doing and then maybe turn off Circle.)
# cron: "10 13-23 * * 1-5"
# filters:
# branches:
# only: main
6 changes: 2 additions & 4 deletions .github/workflows/zoom-upload.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,8 @@ name: Upload Zoom Recordings to YouTube

on:
schedule:
# 10 minutes after every hour before noon on weekdays
# TODO: when we are ready to switch off Circle, run every hour instead of
# just in the morning.
- cron: "10 0-12 * * 1-5"
# 10 minutes after every hour on weekdays
- cron: "10 * * * 1-5"

workflow_dispatch:
inputs:
Expand Down

0 comments on commit 5f59a8c

Please sign in to comment.