Merge pull request #6 from vjcitn/vince

there is a little bug in contains_required
ropensci · Oct 2, 2020 · 35bd510 · 35bd510
2 parents f3a0708 + d78060f
commit 35bd510
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,6 +1,6 @@
 Package: binman
 Title: A Binary Download Manager
-Version: 0.1.1
+Version: 0.1.1-1
 Authors@R: c(
     person("John", "Harrison", , "[email protected]",
     role = "aut", comment = "original author"),

diff --git a/R/assertions.R b/R/assertions.R
@@ -84,7 +84,7 @@ assertthat::on_failure(is_data.frame) <- function(call, env) {
 }
 
 contains_required <- function(x, required){
-  is.list(x) && (required %in% names(x))
+  is.list(x) && all(required %in% names(x))
 }
 
 assertthat::on_failure(contains_required) <- function(call, env) {