search

ga4gh / task-execution-schemas

Apache License 2.0
80 stars 28 forks source link

Fix python syntax #81

Closed giannidallatorre closed 6 years ago

buchanae commented 6 years ago

I think that's JSON, not python, so "false" would be correct.

giannidallatorre commented 6 years ago

Hi Alex,

Oh sorry, you are right I was testing an implantation in Python and I got confused, my bad.

Actually the comma at the end line, of the last element of the lists is not JSON compliant, while it would be with Python!

Cheers, Gianni

On 9 Oct 2017, at 15:56, Alex Buchanan notifications@github.com wrote:

I think that's JSON, not python, so "false" would be correct.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ga4gh/task-execution-schemas/pull/81#issuecomment-335183568, or mute the thread https://github.com/notifications/unsubscribe-auth/ADmehkEMXwx-GZ9hckgeBWypyVdG0qjsks5sqjQ6gaJpZM4PyhCM.

buchanae commented 6 years ago

Good point about the commas, I'll run these through a JSON linter. Thanks!