-
Notifications
You must be signed in to change notification settings - Fork 27
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
Add task for terra backend env exploration #495
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to clarify, this isn't actually a PR against master
, you're just playing in a branch to grab some reconnaissance / telemetry from Terra, but can the PR be closed?
description: "Command that can optionally be included and executed as part of this task" | ||
} | ||
} | ||
command { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
command { | |
command <<< |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
probably need to switch syntax?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we standardizing toward <<<
to discourage WDL interpolation of${}
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes there's definitely a number of bashisms that simply won't work with the older WDL syntax
Yeah, this is all for inspection and not intended to be merged (unless we want to preserve some ability to inspect the environment?) |
Ok, recommend closing this PR, as it's not necessary for the build, CI testing, or Terra testing |
No description provided.