Merge pull request #628 from lorenzwalthert/issue-626

- Remove trainling spaces before processing text (#628).

Author: lorenzwalthert

NEWS.md

@@ -1,3 +1,16 @@
+# styler 1.3.2.9000 (Development)
+
+## Major changes
+
+
+## Minor chnages and fixes
+
+- overhaul pgkdown site: Add search (#623), group function in Reference (#625).
+- always strip trailing spaces and make cache insensitive to it (#626).
+- typos in documentation (#618, #614).
+
+
+
 # styler 1.3.2
 
 Release upon request by the CRAN team.

R/transform-files.R

@@ -80,6 +80,7 @@ make_transformer <- function(transformers,
   assert_transformers(transformers)
 
   function(text) {
+    text <- trimws(text, which = "right")
     should_use_cache <- cache_is_activated()
 
     if (should_use_cache) {

tests/testthat/test-stylerignore.R

@@ -6,15 +6,15 @@ test_that("gives warning markers are not correct", {
   )))
 })
 
-test_that("trailing spaces are stripped when checking marker, but not written back", {
+test_that("trailing spaces are stripped when checking marker and written back", {
   expect_equal(
     style_text(c(
       "# styler: off ",
       "1+1",
       "# styler: on "
     )) %>%
       as.character(),
-    c("# styler: off ", "1+1", "# styler: on")
+    c("# styler: off", "1+1", "# styler: on")
   )
 })