Minor things to fix

[andrew-edwards]

From Sean's Pull Request:

Note there are still errors, warnings, and notes on R CMD check around packages not in imports, is.ggplot as well as

Please use `all_of()` or `any_of()` instead.
  # Was:
  data %>% select(values_from)

  # Now:
  data %>% select(all_of(values_from))

some test warnings and failures, and a bunch of 'no visible binding for global variable' (probably dplyr or ggplot tidy eval) and mismatch in function definitions.

Have to look into why it passes with GHA though.

[seananderson]

It passes on GHA because it's set to only error on error: https://github.com/pbs-assess/pacea/blob/616c4e1b452ada9581bf641e6ac1ec9262fd3b3c/.github/workflows/R-CMD-check.yaml#L33C2-L33C30
vs. warnings + notes.