Skip to content

Commit

Permalink
Merge pull request #503 from ropensci/existcheck
Browse files Browse the repository at this point in the history
Add condition for when no defaults at all are defined
  • Loading branch information
michaelquinn32 authored Oct 27, 2019
2 parents 2170bba + 1613121 commit fb68e6d
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion R/skim_with.R
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,8 @@ validate_assignment <- function(...) {

defaults <- get_default_skimmers()
existing <- proposed_names %in% names(defaults)
if (!all(existing)) {

if (!all(existing) & length(defaults) > 0 ) {
collapsed <- paste(proposed_names[!existing], collapse = ", ")
message(
"Creating new skimming functions for the following classes: ",
Expand Down

0 comments on commit fb68e6d

Please sign in to comment.