broadinstitute / cromwell-tools

A collection of Python clients and accessory scripts for interacting with the Cromwell
https://cromwell-tools.readthedocs.io
BSD 3-Clause "New" or "Revised" License
22 stars 9 forks source link

[WIP] Continued effort for refactoring and standardizing the Cromwell-tools API #39

Closed rexwangcc closed 6 years ago

rexwangcc commented 6 years ago

I still need to fix a lot of unit tests, but the main part is ready for an early review!


Please ensure the following when opening a PR:

samanehsan commented 6 years ago

Is this PR up to date with the master branch? I noticed it is missing this function to validate workflows: https://github.com/broadinstitute/cromwell-tools/blob/master/cromwell_tools/cromwell_tools.py#L438

samanehsan commented 6 years ago

Done with a first-pass review @rexwangcc! :octocat:

rexwangcc commented 6 years ago

Closing with https://github.com/broadinstitute/cromwell-tools/pull/43