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.

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

Add argument to `install()` and `install_deps()` #712

Merged
merged 2 commits into from Apr 28, 2015

Conversation

Projects
None yet
2 participants
@jimhester
Member

jimhester commented Feb 3, 2015

This allows you to pass all arguments to install.packages if needed.

Motivation for this is to be able to set a alternate repo argument for install_deps, for instance to use devtools::install() to automatically install dependencies for a bioconductor package.

install(repos = BiocInstaller::biocinstallRepos(), dependencies = TRUE)
@hadley

This comment has been minimized.

Member

hadley commented Apr 21, 2015

Could you add a note to NEWS.md please?

jimhester added some commits Feb 3, 2015

Add argument to `install()` and `install_deps()`
This allows you to pass all arguments to `install.packages` if needed.

@hadley hadley merged commit dbe0994 into r-lib:master Apr 28, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@hadley

This comment has been minimized.

Member

hadley commented Apr 28, 2015

Thanks!

(FWIW github only notifies on comments, not commits, so it's useful to add a comment like "PTAL" when done so I see the changes sooner)

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