fetch-and-ingest: Explicitly test FETCH_FROM_DATABASE

Prompted by post-merge review #440 (comment) Instead of running `true` and `false` programs, explicitly test that `FETCH_FROM_DATABASE` is true.
nextstrain · Apr 19, 2024 · 6abb9a9 · 6abb9a9
1 parent 512e2ea
commit 6abb9a9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/fetch-and-ingest-genbank-master.yml b/.github/workflows/fetch-and-ingest-genbank-master.yml
@@ -64,7 +64,7 @@ jobs:
         run: |
           config="--config"
 
-          if $FETCH_FROM_DATABASE; then
+          if [[ "$FETCH_FROM_DATABASE" == true ]]; then
             config+=" fetch_from_database=True"
           else
             config+=" fetch_from_database=False"

diff --git a/.github/workflows/fetch-and-ingest-gisaid-master.yml b/.github/workflows/fetch-and-ingest-gisaid-master.yml
@@ -63,7 +63,7 @@ jobs:
         run: |
           config="--config"
 
-          if $FETCH_FROM_DATABASE; then
+          if [[ "$FETCH_FROM_DATABASE" == true ]]; then
             config+=" fetch_from_database=True"
           else
             config+=" fetch_from_database=False"