fix typos

Author: Bai-Li-NOAA

.github/workflows/call-allcontributors.yml

@@ -4,8 +4,10 @@
 name: Collect contributors
 
 on:
-  schedule:
-    - cron: '0 8 * * 1'
+  # schedule:
+  #   - cron: '0 8 * * 1'
+  # TODO: remove push trigger once tests are passing and uncomment schedule
+  push:
   workflow_dispatch:
 
 jobs:

.github/workflows/call-pr-checklist.yml

@@ -3,6 +3,8 @@
 name: Add a comment with reviewer checklist when PR opened
 
 on:
+  # TODO: remove push trigger once tests are passing
+  push:
   pull_request:
     types: [opened]
 
@@ -36,7 +38,7 @@ jobs:
             - [ ] The functionality is good for the users of the code.
             - [ ] Code coverage remains high, indicating the new code is tested.
             - [ ] The code is commented and the comments are clear, useful, and mostly explain why instead of what.
-            - [ ] Code is appropriately documented (doxygen and roxygen).
+            - [ ] Code is appropriately documented.
             `
             const { issue: { number: issue_number }, repo: { owner, repo }  } = context;
             github.issues.createComment({ issue_number, owner, repo, body: msg });

tests/testthat/setup.R

@@ -8,25 +8,12 @@ temp <- scan(base_run_path, what = "", sep = "\n", quiet = TRUE)
 
 # Check if the second element of temp contains "oid" as part of the string.
 # If so, run the system command to install git-lfs and pull the data
-# Code for Linux system
 if (grepl("oid", temp[2])) {
-  # Check if the system is Linux
-  if (Sys.info()["sysname"] != "Linux") {
-    system(
-      "sudo apt update; sudo apt install git-lfs; git lfs install; git lfs pull",
-      intern = TRUE
-    )
-  } 
-
-  # Check if the system is macOS
-  if (Sys.info()["sysname"] == "Darwin") {
-    system(
-      "brew install git-lfs; git lfs install; git lfs pull",
-      intern = TRUE
-    )
-  }
-
-  if (Sys.info()["sysname"] == "Windows") {
-    system("winget install -e --id GitHub.GitLFS; git lfs install; git lfs pull")
-  }
-}
+  sysname <- Sys.info()[["sysname"]]
+  switch(
+    sysname,
+    "Linux" = system("sudo apt update; sudo apt install -y git-lfs; git lfs install; git lfs pull", intern = TRUE),
+    "Darwin" = system("brew install git-lfs; git lfs install; git lfs pull", intern = TRUE),
+    "Windows" = system("winget install -e --id GitHub.GitLFS; git lfs install; git lfs pull", intern = TRUE)
+  )
+}