update to cli_warn

pharmaverse · Sep 30, 2024 · 5ab8252 · 5ab8252
1 parent e846b67
commit 5ab8252
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/R/derive_vars_cat.R b/R/derive_vars_cat.R
@@ -217,10 +217,9 @@ derive_vars_cat <- function(dataset,
 
   # warn if new variables already exist
   if (any(new_col_names %in% names(dataset))) {
-    warning(paste("Column(s) in `definition` already exist in `dataset`.",
-      "Did you forget to specify `by_vars`,",
-      "or are you rerunning your code?",
-      sep = "\n"
+    cli_warn(paste("Column(s) in {.arg definition} already exist in {.arg dataset}.",
+      "Did you forget to specify {.arg by_vars},",
+      "or are you rerunning your code?"
     ))
   }
 

diff --git a/tests/testthat/_snaps/derive_vars_cat.md b/tests/testthat/_snaps/derive_vars_cat.md
@@ -1,8 +1,6 @@
 # derive_vars_cat Test 3: Forgot to specify by_vars
 
-    Column(s) in `definition` already exist in `dataset`.
-    Did you forget to specify `by_vars`,
-    or are you rerunning your code?
+    Column(s) in `definition` already exist in `dataset`. Did you forget to specify `by_vars`, or are you rerunning your code?
 
 # derive_vars_cat Test 13: definition has wrong shape