-
Notifications
You must be signed in to change notification settings - Fork 120
function used to import an invocation archive. #506
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
base: main
Are you sure you want to change the base?
function used to import an invocation archive. #506
Conversation
…re it can be merged.
e347c4e
to
ee710a4
Compare
…reading it into memory
For this to work, we need the following galaxy pull-request 20359 to be merged. |
@nsoranzo should I add some comment to the documentation that this function requires version XXX of galaxy? |
@@ -112,6 +112,18 @@ def test_get_invocation_jobs_summary(self): | |||
assert len(step_jobs_summary) == 1 | |||
assert step_jobs_summary[0]["populated_state"] == "ok" | |||
|
|||
@test_util.skip_unless_galaxy("release_25.01") |
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.
@test_util.skip_unless_galaxy("release_25.01") | |
@test_util.skip_unless_galaxy("release_25.1") |
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.
I will wait with adding this until I know at which release the dependent code change end up on.
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.
Sure, just noting that's the format we are currently using for releases.
Yes, please, that would be helpful! Example: bioblend/bioblend/galaxy/users/__init__.py Lines 226 to 228 in 8e08b05
|
Needs API updates before it can be merged.