-
Notifications
You must be signed in to change notification settings - Fork 70
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 explicit Python requirements to tools that appear to require it #142
Conversation
@@ -10,6 +10,7 @@ | |||
<xml name="preamble"> | |||
<requirements> | |||
<requirement type="package" version="@TOOL_VERSION@">blast</requirement> | |||
<yield /> |
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.
Yield?
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.
<expand macro="preamble" /> | ||
<expand macro="preamble"> | ||
<requirement type="package" version="3.9">python</requirement> | ||
</expand> |
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 macros can do that can they?
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.
That extended macro stuff is rather neat.
I really ought to get CI working here again though...
Thanks Peter! |
Mh, it seems this was never deployed. I don't see 0.1.2 of blastxml_to_top_descr.xml in the TS. |
Looks like it was done between TravisCI and GitHub Actions automation... |
I have updated the tool sheds manually for https://github.com/natefoo/galaxy_blast/tree/master/tools/blastxml_to_top_descr |
No description provided.