Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove the one-space indentation of last line #6077

Merged
merged 3 commits into from
Aug 13, 2019

Conversation

sclan
Copy link
Contributor

@sclan sclan commented Aug 2, 2019

Reason: With the indentation before the closing bracket, the task Funcotate is ignored by tools such as dxWDL.

Reason: With the indentation before the closing bracket, the task Funcotate is ignored.
Copy link
Member

@lbergelson lbergelson left a comment

Choose a reason for hiding this comment

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

No objections to this change. It's my understanding that whitespace is meant to be ignored in wdl though. It seems like whatever tool is parsing this has a bug and you might want to file a bug report with them.

@lbergelson
Copy link
Member

We're finally out of our test failure nightmare. It looks like someone might have deleted the space in the interim, but I'll merge this anyway in case it helps. Thank you @sclan

@lbergelson lbergelson merged commit 4b55556 into broadinstitute:master Aug 13, 2019
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