Get and check aes params about ggcheck HOT 6 CLOSED

I think it would be easier if it was one function, but it does mess with the naming convention a bit since fill is not technically a geom param.

It would also be nice if the function used ... so you don't have to put the parameters in a list (like what you did with the labels functions).

from ggcheck.

Would you envision this as one function that can check both geom params and aes params, like uses_geom_param(p, "bar", params = list(fill = "blue")), or a new function to check aes params, like uses_aes_param()?

from ggcheck.

Looking at the documentation, it seems that the function is already a bit overloaded. It currently checks for geom_params and stat_params, so I don't think adding aes_params would cause much issue.

from ggcheck.

@skaltman Currently, uses_labels() returns a named vector if you check more than one label. uses_geom_params() returns a single logical value indicating if all params match. Should uses_geom_params() return a named vector as well?

library(ggcheck)
library(ggplot2)

p <- ggplot(mpg, aes(x = manufacturer, y = hwy)) +
  geom_boxplot(width = 0.5, na.rm = TRUE) +
  labs(x = "Manufacturer", y = "MPG")

uses_labels(p, x = "Manufacturer", y = "MPG")
#>    x    y 
#> TRUE TRUE
uses_geom_param(p, "boxplot", width = 0.5, na.rm = TRUE)
#> [1] TRUE

^{Created on 2021-11-30 by the reprex package (v2.0.1)}

from ggcheck.

@rossellhayes Yeah, I think a named vector would work well. It seems like it would be good if both functions worked similarly. And if uses_geom_param() returns a named vector, you can check where exactly the student code went wrong without writing multiple calls to uses_geom_param().

from ggcheck.

@skaltman Thank you! The PR should now be ready to go, let me know how it looks!

from ggcheck.