Skip to content

Comments

Reversed set assert message for clarity when dealing with long sets a…#286

Open
spgarbet wants to merge 1 commit intomllg:masterfrom
spgarbet:master
Open

Reversed set assert message for clarity when dealing with long sets a…#286
spgarbet wants to merge 1 commit intomllg:masterfrom
spgarbet:master

Conversation

@spgarbet
Copy link

When dealing with sets with a lot of elements, the error message gets truncated and the most important piece of information to the user is lost. This patch fixes that.

Reproducible example:

data <- data.frame(x = 1:5, X = TRUE)
myset <- c(letters, outer(letters, letters, paste0))
coll <- checkmate::makeAssertCollection()
checkmate::assert_subset(x = names(data), 
                         choices = myset, 
                         add = coll)
checkmate::reportAssertions(coll)

vubiostat/redcapAPI#471

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant