new example for vignette about assertr HOT 7 CLOSED

Thanks for the kind words! That would be super useful. Because of the nested magrittr pipes, do you have any idea what exists would look like?

from assertr.

Exists already exists! If you try it, you'll see it works with the current version of assertr : D

from assertr.

Just checked this. It will return TRUE is there's a variable named the same name as the column to be checked...

mpgg <- "this"
mtcars %>%
  verify("mpgg" %>% exists) %>%
  head # passes

Which can be really dangerous.
So you have to do this instead

mpgg <- "this"
mtcars %>%
  verify(exists("mpgg", where=.)) %>%
  head # fails

I think it might be worth writing a dedicated wrapper function (I'll do it!).
What do you think?

from assertr.

Nice catch, yes, in this case a wrapper would be helpful.

On Sat, Aug 15, 2015 at 11:48 AM, Tony Fischetti [email protected]
wrote:

Just checked this. It will return TRUE is there's a variable named the
same name as the column to be checked...

mpgg <- "this"mtcars %>%
verify("mpgg" %>% exists) %>%
head # passes

Which can be really dangerous.
So you have to do this instead

mpgg <- "this"mtcars %>%
verify(exists("mpgg", where=.)) %>%
head # fails

I think it might be worth writing a dedicated wrapper function (I'll do
it!).
What do you think?

—
Reply to this email directly or view it on GitHub
#23 (comment)
.

from assertr.

Because I don't have any other "predicates" that can't be used with verify and not the other verbs I don't think I'm gonna write the wrapper function, but I'm gonna put it in the vignette!

from assertr.

I decided to implement this after all! It's has_all_names :) Thanks for the suggestion!

It's in the README and vignette now, too

from assertr.

👍

from assertr.