-
Notifications
You must be signed in to change notification settings - Fork 10
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
port Bowtie2Index to q2-types #16
Conversation
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.
LGTM, one minor tweak requested - inline. Otherwise, I tested these changes locally, all appears well in q2-shogun land.
@@ -17,7 +17,7 @@ | |||
from qiime2.util import duplicate | |||
from q2_types.feature_data import DNAFASTAFormat | |||
|
|||
from q2_shogun._formats import Bowtie2IndexDirFmt | |||
from q2_types.bowtie2 import Bowtie2IndexDirFmt |
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.
The plugin recipe is missing an entry for q2_types (it also appears to be missing an entry for the framework, too).
thanks @thermokarst ! Added those items to the recipe... 🏀 |
Pull Request Test Coverage Report for Build 35
💛 - Coveralls |
Just moving this to q2-types, nothing exciting to see here
DO NOT MERGE pending this PR: qiime2/q2-types#243