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

Use the proper Bioconductor URL for both release and devel #937

Merged
merged 3 commits into from Jan 12, 2016

Conversation

Projects
None yet
2 participants
@jimhester
Member

jimhester commented Sep 24, 2015

Depending on if the user has ran BiocInstaller::useDevel() the
repository URL differs. The previous behavior was hard-coded to look only
at Bioconductor package in the last release. Making URL a parameter
ensures that if the user changes from looking at devel to release in the
same session the previous cached result will not be used.

jimhester added some commits Sep 24, 2015

Use the proper Bioconductor URL for both release and devel
Depending on if the user has ran BiocInstaller::useDevel() the
repository URL differs. The previous behavior was hard-coded to look only
at Bioconductor package in the last release. Making URL a parameter
ensures that if the user changes from looking at devel to release in the
same session the previous cached result will not be used.
@jimhester

This comment has been minimized.

Member

jimhester commented Jan 11, 2016

This should now be ready to merge.

hadley added a commit that referenced this pull request Jan 12, 2016

Merge pull request #937 from jimhester/bioc_packages
Use the proper Bioconductor URL for both release and devel

@hadley hadley merged commit 2fc4f84 into r-lib:master Jan 12, 2016

1 of 2 checks passed

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

This comment has been minimized.

Member

hadley commented Jan 12, 2016

Thanks!

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