Enabling lintr test on push

Plant-Food-Research-Open · Jan 11, 2024 · e98142f · e98142f
1 parent 56a256c
commit e98142f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/.github/workflows/lint-changed-files.yaml b/.github/workflows/lint-changed-files.yaml
@@ -1,6 +1,8 @@
 # Workflow derived from https://github.com/r-lib/actions/tree/v2/examples
 # Need help debugging build failures? Start at https://github.com/r-lib/actions#where-to-find-help
 on:
+  push:
+    branches: [main, master]
   pull_request:
     branches: [main, master]
 

diff --git a/R/data_import.R b/R/data_import.R
@@ -652,7 +652,7 @@ import_smetadata_csv_factory <- function(files,
 
   res <- lapply(df, function(x) { ## apply to each column
     if (is.character(x)) {
-      ## Detect if any non-ascii characters in column
+      ## Detect if any non-ascii characters
       non_ascii_indx <- which(stringr::str_detect(x, "[^[:ascii:]]"))
 
       if (length(non_ascii_indx)) {