New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove a few plyr calls + other cleanups #509

Open

olivroy wants to merge 6 commits into ggobi:master from olivroy:plyr

olivroy commented Jan 28, 2025 •

edited

Loading

Use pkgdown bootstrap 5 + update links (this adds a search bar)
Use rlang::check_installed() to check missing suggests
Get rid of internal ifNull() to use %||% instead
get rid of a few {plyr} calls
Use recent {scales} function names.
Use .data$ as recommended in https://ggplot2.tidyverse.org/articles/ggplot2-in-packages.html#using-aes-and-vars-in-a-package-function

olivroy added 6 commits

January 28, 2025 13:31


          Get rid of ifNull() and use %||% instead

400ca2a


          Use news {scales} fn names.

e77034c


          Update docs

09fd440


          Get rid of a few plyr calls

75bd0bc


          Use rlang::check_installed() and remove require_namespaces()

233033e


          Extra .data$

24e6b2c

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

Labels

None yet