tabyl() should further clean variable names that it returns about janitor HOT 4 CLOSED

Eh, it's unclear. If you're doing further operations on the result, like a join, this is an advantage. But, you should call clean_names() now, or should have called it before tabyl(). So it's kind of hand-holdy.

If you're not doing further operations, and are just printing to console, then the status quo is more useful in that it confirms what you did. And if you're printing it in a markdown table, you'll probably need to tweak the name anyway.

It's easy to add a clean_names call to the tabyl() function but I'm not sure it's worth it.

from janitor.

I'm fine with retaining the dirty name, like:

mtcars %>% mutate(`bad name` = cyl) -> k
tabyl(k$`bad name`)

But unfortunate that this:

mtcars %>% mutate(`bad name` = cyl) %>% tabyl(`bad name`)

Gives "bad.name" instead of "bad name".

from janitor.

Oh and this behavior (default and piped) should match that of crosstab()

from janitor.

Fixed - now piped calls to both functions will return exactly the variable name, with whatever warts it has, and default calls will return the data.frame name and $, e.g., mtcars$cyl or mtcars$bad name``.

from janitor.