Skip to content

Support directories full of workflows#27

Merged
afgane merged 6 commits intomasterfrom
wf_extend
Apr 3, 2017
Merged

Support directories full of workflows#27
afgane merged 6 commits intomasterfrom
wf_extend

Conversation

@bgruening
Copy link
Copy Markdown
Member

No description provided.

@bgruening bgruening changed the title WIP: Support directories full of workflows Support directories full of workflows Mar 31, 2017
@bgruening bgruening requested a review from jmchilton March 31, 2017 23:25
@bgruening
Copy link
Copy Markdown
Member Author

ok, this one is also ready to go
Could we have a new release after this is merged?

from bioblend import galaxy


def import_workflow(gi, path):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing docstring.

dest="galaxy_url",
help="Target Galaxy instance URL/IP address (required "
"if not defined in the tools list file)",)
parser.add_argument("-a", "--apikey",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not leave both options?

@bgruening
Copy link
Copy Markdown
Member Author

@afgane thanks for the review. I added the docstring and the API option back.

@afgane afgane merged commit 214d59d into master Apr 3, 2017
@afgane
Copy link
Copy Markdown
Contributor

afgane commented Apr 3, 2017

Thanks for the updates @bgruening

@bgruening
Copy link
Copy Markdown
Member Author

Thanks @afgane!

@bgruening bgruening deleted the wf_extend branch April 4, 2017 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants