[FEATURE] Default to version = packageVersion(pkg) and type = "cran"

[ColinFay]

• What are you trying to do?

> oysteR::audit("shiny")
Error in oysteR::audit("shiny") : 
  argument "version" is missing, with no default

> oysteR::audit("shiny", version = packageVersion("shiny"))
Error in generate_purls(pkg, version, type) : 
  argument "type" is missing, with no default

> oysteR::audit("shiny", version = packageVersion("shiny"), type = "cran")
ℹ Using cached results for 1 package
Error: Can't combine `version` <character> and `version` <package_version>.

• What feature or behavior is this required for?

It would be nice to default to version = packageVersion(pkg) and type = "cran", which seems to be the most common use case (auditing package version installed on the machine, and for CRAN).

• How could we solve this issue? (Not knowing is okay!)

Happy to PR this.

• Anything else?

cc @bhamail / @DarthHater / @brittanybelle / @adrianpowell / @csgillespie

[csgillespie]

Thanks @ColinFay

1. Doing a behind the scenes conversion of packageVersion to character seems like a good idea. Just test for class, then convert.
2. I've been unsure about defaults because the API works for other languages

audit("pandas", version = "0.1.0", type = "pypi")

3. My gut feeling is to have a new function: audit_r_pkg that defaults to CRAN, and tests all dependencies as well.

Thoughts?

[ColinFay]

yeah I suppose that does make sense to have an audit_r_pkg with these default :)

Will draft a PR if that's ok with you !

[csgillespie]

That would be great! Thanks Dr Colin Gillespie http://www.mas.ncl.ac.uk/~ncsg3/

…

On Tue, 13 Jul 2021 at 14:57, Colin Fay ***@***.***> wrote: yeah I suppose that does make sense to have an audit_r_pkg with these default :) Will draft a PR if that's ok with you ! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#66 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAJVQI7PZOV2H7ELGM54Y43TXRA5DANCNFSM5AIQ4K2A> .