-
Notifications
You must be signed in to change notification settings - Fork 9
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
Update to 1.16 #32
Update to 1.16 #32
Conversation
@@ -1,6 +1,6 @@ | |||
import click | |||
from parsec.cli import pass_context, json_loads | |||
from parsec.decorators import custom_exception, list_output | |||
from parsec.decorators import custom_exception, dict_output |
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.
hmm, this looks wrong, I wonder what happened here.
it's definitely list[dict]
https://bioblend.readthedocs.io/en/latest/api_docs/galaxy/all.html?highlight=get_datasets#bioblend.galaxy.datasets.DatasetClient.get_datasets
Thanks for doing this @simonbray! This looks good overall, but clearly need to figure out what went wrong with the output type detection. I'll look into it. |
Fantastic. Thanks for doing the update @simonbray! |
It's live. Also migrated it to github actions since travis is travis. |
Nice. Thanks a lot @hexylena! |
The autobuilder script made a few strange changes, especially adding a lot of
module.py
files, which I reverted. Overall it was a very smooth process though.