-
Notifications
You must be signed in to change notification settings - Fork 3
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 support for Bioconductor #58 #59
Merged
Merged
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
1c6331b
added bioc versions
schochastics 8c34a79
fixed ordering error in cahced_biocver
schochastics bd37ca5
added query_biocver
schochastics 0907a90
added bioc_package_history
schochastics 160d381
cached_biocver to data.frame
schochastics 397c891
fixed missing supported_os
schochastics 80215bb
added biocver generation
schochastics 7f49b93
check if pkg is bioc
schochastics 2c4da10
fix indention
schochastics dce1271
added is.bioc and adjusted normalize_pkg
schochastics db5b2f2
added query_snapshot_dependencies_bioc
schochastics bceff5a
finalized resolve
schochastics 2a4feb9
Merge remote-tracking branch 'upstream/v0.2' into bioc
schochastics bcab080
added bioc support for export_rang
schochastics 06564a1
probably fixed sysreqs for bioc
schochastics 93ece47
fixed sysreq return value
schochastics 9f3cd88
fixed typo
schochastics bbdde48
added resolve test for bioc
schochastics 37defb3
added test for bioc version
schochastics 957fc7d
missed the dot
schochastics 9709762
added test for new bioc release
schochastics 5a2c69d
test dockerize with bioc
schochastics 04c70ad
better bioc test data
schochastics e5997a2
added package caching for bioc
schochastics 29e3b7f
set query_bioc TRUE if bioc detected
schochastics 1a04a3b
added cache test for bioc
schochastics 0c5d1d1
cache testing,fixed header/footer, always write bioc version
schochastics 15ab0a0
integrated bioc into as_pkgref
schochastics b723ed6
get bioc versions from online
schochastics File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@chainsawriot This function requires the bioc version to check if the package is in a specific release. Couldnt find any other way to check this thus far, but this would require to also add bioc_version as a parameter to .normalize_pkg
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestion for
.normalize_pkg()
: make bioc_version optional. If it is omitted, it is assumed to be a cran package. if present, check