fix: allow both unstaged, e.g. for pre-commit run --all-files

Author: lorenzwalthert

inst/hooks/exported/readme-rmd-rendered.R

@@ -35,7 +35,9 @@ if (file.exists("README.md")) {
   source_staged <- source_file %in% file_names_staged
   output_staged <- "README.md" %in% file_names_staged
 
-  if (!source_staged || !output_staged) {
-    rlang::abort(paste(source_file, "and README.md must both be staged."))
+  if (sum(source_staged, output_staged) == 1) {
+    rlang::abort(paste(
+      source_file, "and README.md must both be either staged or unstaged."
+    ))
   }
 }

tests/testthat/test-hook-readme-rmd-rendered.R

@@ -47,7 +47,7 @@ if (has_git()) {
     run_test("readme-rmd-rendered",
       file_name = c("README.Rmd", "README.md"),
       suffix = "",
-      std_err = "must both be staged",
+      std_err = "must both be either staged",
       std_out = NULL,
       file_transformer = function(files) {
         if (length(files) > 1) {
@@ -112,7 +112,7 @@ if (has_git()) {
     run_test("readme-rmd-rendered",
       file_name = c("README.qmd", "README.md"),
       suffix = "",
-      std_err = "must both be staged",
+      std_err = "must both be either staged",
       std_out = NULL,
       file_transformer = function(files) {
         if (length(files) > 1) {