Workflow cleanup (#38)

* Switch update to default on
* Remove update option from workflow_dispatch
* Add missing dep

Signed-off-by: Phil Dibowitz <phil@ipom.com>

Author: jaymzh

.github/workflows/sync.yml

@@ -4,12 +4,6 @@ on:
   schedule:
     - cron: '1,31 * * * *' # twice an hour
   workflow_dispatch:
-    inputs:
-      update:
-        description: 'Run sync with --update to force all sessions to be updated'
-        type: boolean
-        required: false
-        default: false
 
 jobs:
   sync:
@@ -33,11 +27,7 @@ jobs:
           GUIDEBOOK_API_TOKEN: ${{ secrets.GUIDEBOOK_API_TOKEN }}
           GUIDEBOOK_JWT_TOKEN: ${{ secrets.GUIDEBOOK_JWT_TOKEN }}
         run: |
-          CMD="python ./guidebook/sync_guidebook.py --max-deletes 5 -u"
-
-          if [[ "${{ github.event.inputs.update }}" == "true" ]]; then
-            CMD="$CMD --update"
-          fi
+          CMD="python ./guidebook/sync_guidebook.py --max-deletes 5"
 
           echo "Running: $CMD"
           $CMD

guidebook/requirements.txt

@@ -2,3 +2,4 @@ click
 requests
 pytz
 xdg-base-dirs
+dateutil

guidebook/sync_guidebook.py

@@ -764,8 +764,8 @@ def get_tokens(logger):
 @click.option(
     "--update/--no-update",
     "-u",
-    default=False,
-    help="Update existing sessions.",
+    default=True,
+    help="Update existing sessions. Does so idempotently, defaults to True.",
 )
 @click.option(
     "--delete-all/--no-delete-all",