-
Notifications
You must be signed in to change notification settings - Fork 57
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
regex find/replace: use format_source for output #679
Conversation
Version 1.0.2 of the tool uses format="input" in its output declaration. This creates an output with data type "input". I figure, the intention is instead to create an output with the same data type as the input. This should fix it, right? Incidentally, this same error can currently be found in the Tool XML documentation.
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.
Looks good to me!
@bernt-matthias good to go?
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.
format_source
is more explicit (I remember that format="input"
implies that the format is taken from the 1st input).
Can you add the ftype
attribute to the in- and outputs in the tests.
Got it: in the last PR we set a profile. Problem is that Using format_source is the correct fix here. |
stumbled over this here: galaxyproteomics/tools-galaxyp#679 could not find a test
stumbled over this here: galaxyproteomics/tools-galaxyp#679 could not find a test
stumbled over this here: galaxyproteomics/tools-galaxyp#679 could not find a test
Version 1.0.2 of the tool uses format="input" in its output declaration. This creates an output with data type "input". I figure, the intention is instead to create an output with the same data type as the input. This should fix it, right?
Incidentally, this same error can currently be found in the Tool XML documentation.