Skip to content

Commit

Permalink
Remove this comment
Browse files Browse the repository at this point in the history
  • Loading branch information
jennybc committed Apr 30, 2023
1 parent 7444751 commit 0e307b0
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions testing-advanced.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -380,8 +380,6 @@ skip_on_os("windows")
One challenge with skips is that they are currently completely invisible in CI --- if you automatically skip too many tests, it's easy to fool yourself that all your tests are passing when in fact they're just being skipped!
In an ideal world, your CI/CD would make it easy to see how many tests are being skipped and how that changes over time.

*2022-06-01: Recent changes to GitHub Actions mean that we will likely have better test reporting before the second edition of this book is published. Stay tuned!*

It's a good practice to regularly dig into the `R CMD check` results, especially on CI, and make sure the skips are as you expect.
But this tends to be something you have to learn through experience.

Expand Down

0 comments on commit 0e307b0

Please sign in to comment.