search

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

missing comma in setup.py ? #84

Closed notestaff closed 5 years ago

notestaff commented 5 years ago

at https://github.com/broadinstitute/cromwell-tools/blame/master/setup.py#L19 seems a comma is missing? @rexwangcc @samanehsan

rexwangcc commented 5 years ago

@notestaff Thanks, this is a good catch. Luckily that extra comma didn't affect the installation of google-auth but '\setuptools_scm which influence the development only and somehow covered by setup_requires=['setuptools_scm']. While I'm wondering why it was not captured by the linters, a quick fix should be ready shortly.

Thanks again!