Skip to content

Commit

Permalink
adjust documentation for skip_vdiffr
Browse files Browse the repository at this point in the history
 - This environment variable is no longer set in CI. Based on previous findings, this was necessary to avoid differences in SVG/snapshot expectation when installing various versions of `vdiffr` in CI.

 - This doesnt seem to be necessary now. The reasoning is unclear, but opting to _not_ set the environment variable in `.github/workflows/main.yaml` unless necessary.
  • Loading branch information
barrettk committed Aug 8, 2024
1 parent 2f2cbd8 commit 31469fb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion R/helpers-for-tests.R
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#' Skip test if not on Metworx or CI
#' Skip test if `PMF_SKIP_VDIFFR` environment variable set to `TRUE`
#'
#' Checks for custom `PMF_SKIP_VDIFFR` environment variable and skips any tests
#' that use `vdiffr` if set to `"true"`. This is useful because
Expand Down
2 changes: 1 addition & 1 deletion man/skip_vdiffr.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 31469fb

Please sign in to comment.