-
Notifications
You must be signed in to change notification settings - Fork 991
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
Revise tool parameter handling, remove redundancies #1711
Conversation
4a593aa
to
e795b9d
Compare
@galaxybot test this |
Looks like there are some legitimate failures in the test cases, if the problem just becomes test cases using deprecated features though let me know I can update the tests. |
85112c4
to
bd6c1fe
Compare
Looks good to me: +1 |
No worries and thanks a lot for the comments. |
579d6ce
to
f341ca3
Compare
f341ca3
to
5041c80
Compare
I tested composite file uploads within a docker image and i did not notice anything odd. |
Thanks this is a substantial cleanup and simplification of various complex things - this should set the stage for awesome things down the road. |
Revise tool parameter handling, remove redundancies
@guerler it is merged, bot will ignore it I think |
This is a continuation of #1572 and #115 and our ongoing effort to unify the tool parameter handling, remove hacks, legacy handling and redundancies.
https://github.com/galaxyproject/galaxy/pull/1711/files#diff-1bef786a2b2ea704c1748e41cd25dd33L2781
https://github.com/galaxyproject/galaxy/pull/1711/files#diff-c2872901d34ca0c09ded74872356dd7bL14
https://github.com/galaxyproject/galaxy/pull/1711/files#diff-c2872901d34ca0c09ded74872356dd7bR60
https://github.com/galaxyproject/galaxy/pull/1711/files#diff-9baf995401cfeb779edf8731ebaf0d2dL1649
https://github.com/galaxyproject/galaxy/pull/1711/files#diff-5cab48d5f01fec3fa30d17f4bf5d1848L1048
guerler@050adad