-
Notifications
You must be signed in to change notification settings - Fork 17
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 jq and curl #23
Add jq and curl #23
Conversation
Why does this CLI image need those utilities? How would people use them in an actual CI workflow? |
The action is also useful for running more complex scripts that assume the caller is already authenticated with cloud.gov and pointing to the appropriate space. Here's an example workflow and the script it points to. In this case the script is mainly using |
I would also like to add
We have a workflow step to monitor the output of a cf command, in which a script is relying on grep's Is it possible to add |
I'm okay with that; @markdboyd ...? Also if there's a common pack of binutil-type stuff not covered by BusyBox, let's just go ahead and install it. (And: Sorry for the breaking change, @FuhuXia !) |
@mogul sure, feel free to add |
Changes proposed in this pull request:
Security considerations
None, these are well-understood CLI utilities that are being pulled from the upstream Alpine repository, which is updated frequently.