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
docs: Add example of container
step standard output as parameter. Fixes #12961
#12963
base: main
Are you sure you want to change the base?
Conversation
…er (argoproj#12961) Signed-off-by: Sandeep Khurana <skhurana333@gmail.com>
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.
The documentation does not has example of showing stdout of a step as output parameter.
This is not correct, there is an example on an adjacent page: https://argo-workflows.readthedocs.io/en/latest/walk-through/scripts-and-results/.
As such this seems redundant, and in its current state, explains less than the existing example. It also re-introduces result
without a description.
at https://argo-workflows.readthedocs.io/en/latest/walk-through/output-parameters/ c) results is already mentioned in same url above |
container
step standard output as parameter. Fixes #12961
That rationale makes more sense -- you didn't mention the Otherwise, see my comment in the issue, we may want to move the entire "result" section out of this page and into the other page (including your example). |
Fixes #12961
Motivation
The documentation does not has example of showing stdout of a step as output parameter.
Modifications
An example has been added.
Verification
Ran the example, its working fine and produced expected result.