Merge pull request #374 from insightsengineering/fix-pipes

updated news / description
insightsengineering · Oct 18, 2022 · dc4dbdd · dc4dbdd
2 parents 107af83 + 37af433
commit dc4dbdd
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,6 +1,6 @@
 Package: rbmi
 Title: Reference Based Multiple Imputation
-Version: 1.2.0
+Version: 1.2.1
 Authors@R: c(
     person("Craig", "Gower-Page", email = "[email protected]", role = c("aut", "cre")),
     person("Alessandro", "Noci", email = "[email protected]", role = c("aut")),

diff --git a/NEWS.md b/NEWS.md
@@ -2,6 +2,10 @@
 
 * No updates yet
 
+# rbmi 1.2.1
+
+* Removed native pipes `|>` in testing code so package is backwards compatible with older servers
+
 # rbmi 1.2.0
 
 * Replaced our `glmmTMB` dependency with the `mmrm` package. This has resulted in the package being more stable (less model fitting convergence issues) as well as speeding up run times 3-fold. 

diff --git a/tests/testthat/test-mmrm.R b/tests/testthat/test-mmrm.R
@@ -466,7 +466,7 @@ test_that("visit & group factor levels / order doesn't break model extraction",
 
 
 
-    dat_modified <- dat |>
+    dat_modified <- dat %>%
         mutate(group = relevel(group, "B"))
 
     mod <- mmrm::mmrm(
@@ -481,7 +481,7 @@ test_that("visit & group factor levels / order doesn't break model extraction",
 
 
 
-    dat_modified <- dat |>
+    dat_modified <- dat %>%
         mutate(visit = relevel(visit, "visit_3"))
 
     mod <- mmrm::mmrm(