Skip to content

Also update data managers#78

Merged
rhpvorderman merged 16 commits intogalaxyproject:masterfrom
rhpvorderman:get_data_managers
Mar 13, 2018
Merged

Also update data managers#78
rhpvorderman merged 16 commits intogalaxyproject:masterfrom
rhpvorderman:get_data_managers

Conversation

@rhpvorderman
Copy link
Copy Markdown
Contributor

@rhpvorderman rhpvorderman commented Mar 6, 2018

Hi. This solves #69 .
Have not run extensive testing yet, but will do that tomorrow.
It seems to run for now. Curious to see your feedback.

One of the side effects is that get-tool-list now also uses get_galaxy_connection method. a nice step towards unifying ephemeris.

EDIT:
Changes:

  • shed-tools update now also updates data managers.
  • With get-tool-list you can now also get all installed data managers, provided you have admin access to the galaxy.

@jmchilton
Copy link
Copy Markdown
Member

Very nice - 👍 from me. Thanks a bunch @rhpvorderman !

Comment thread ephemeris/get_tool_list_from_galaxy.py Outdated
"Use this if you would like to use the list to update all the tools in"
"your galaxy instance using shed-install."
)
parser.add_argument("-get_dms", "--get_data_managers",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Imho the single dashes are meant for single letter. I would skip the short version here.

@galaxyproject galaxyproject deleted a comment Mar 13, 2018
@rhpvorderman rhpvorderman merged commit 861a98a into galaxyproject:master Mar 13, 2018
@rhpvorderman rhpvorderman deleted the get_data_managers branch March 13, 2018 08:44
@rhpvorderman
Copy link
Copy Markdown
Contributor Author

@bgruening @jmchilton Thanks for reviewing!

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.

3 participants