DominikRafacz/deepdep

Package Version Argument

jessecambon opened this issue · 3 comments

jessecambon commented

First of all thanks for this package, it's very useful. I am not sure how feasible this would be to implement, but it could be helpful to have a package version argument for the get_dependencies and deepdep functions. For my use case, I was trying to compare the dependencies of an old version of my package to the current version.

One workaround I attempted was to install the old version of the package with devtools::install_version and then use deepdep with local = TRUE. However, I ran into an issue installing the old package version so this didn't quite work.

ErdaradunGaztea commented

Getting around to doing this now, since I just discovered that pkgsearch package has exactly the functionality we need (for CRAN at least, Bioc is a separate animal).

  • 🎉1
ErdaradunGaztea commented

TODO:

  • Store cached description in a list of versions inside a list of packages instead as a list of packages.
  • Keep info about version for description printing and other uses (e.g. plotting).
  • Add test coverage obviously.
ErdaradunGaztea commented

Just realized that pkgsearch uses curl to make HTTP calls, which... complicates testing beyond reason, since we wouldn't be able to use vcr anymore. It will be better to reimplement pkgsearch functionality instead with httr, just like in current version of deepdep.

  • 😕1