Merge pull request #127 from MichaelChirico/patch-1

drop plyr::empty usage
tonyfischetti · Nov 14, 2023 · 1636b6d · 1636b6d
2 parents 8b04b89 + 12c2ff0
commit 1636b6d
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 14 deletions.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -29,7 +29,6 @@ Suggests:
  magrittr,
  rmarkdown,
  tibble,
- plyr,
  covr
 VignetteBuilder: knitr
 RoxygenNote: 7.2.3

diff --git a/codemeta.json b/codemeta.json
@@ -97,18 +97,6 @@
  "url": "https://cran.r-project.org"
  },
  "sameAs": "https://CRAN.R-project.org/package=tibble"
- },
- {
- "@type": "SoftwareApplication",
- "identifier": "plyr",
- "name": "plyr",
- "provider": {
- "@id": "https://cran.r-project.org",
- "@type": "Organization",
- "name": "Comprehensive R Archive Network (CRAN)",
- "url": "https://cran.r-project.org"
- },
- "sameAs": "https://CRAN.R-project.org/package=plyr"
  }
  ],
  "softwareRequirements": {

diff --git a/tests/testthat/test-assertions.R b/tests/testthat/test-assertions.R
@@ -1610,6 +1610,6 @@ test_that("handle predicates applied to the whole data, and not to subframe", {
  head( n = 0)
 
  expect_silent({
- assert(data = a_tibble, predicate = plyr::empty, dplyr::everything())
+ assert(data = a_tibble, predicate = function(x) nrow(x) == 0L, dplyr::everything())
  })
 })