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

docs(cli): add example of list label value #13707

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

D0ri123
Copy link

@D0ri123 D0ri123 commented Oct 5, 2024

Fixes #11898

Motivation
Modify the argo archive list-label-values command for more info.

Modifications
Added examples for the argo archive list-label-values command.

Verification
run argo archive list-label-values -h

@D0ri123 D0ri123 force-pushed the docs/archive-command-list-label-value-example--help-ouput branch from f05c9e7 to 6824c01 Compare October 5, 2024 11:55
@D0ri123
Copy link
Author

D0ri123 commented Oct 6, 2024

@agilgur5 this PR is failing in the ci-build/Codegen step. How can I resolve this issue?

@@ -18,6 +18,9 @@ func NewListLabelValueCommand() *cobra.Command {
command := &cobra.Command{
Use: "list-label-values",
Short: "get workflow label values in the archive",
Example: `# Get workflow label values in the archive:
argo archive list-label-values -l key1
`,
Copy link
Member

@leehosu leehosu Oct 6, 2024

Choose a reason for hiding this comment

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

이 부분에 공백을 확인해보세요!

@D0ri123 D0ri123 force-pushed the docs/archive-command-list-label-value-example--help-ouput branch 2 times, most recently from 149392b to 6824c01 Compare October 13, 2024 09:16
@D0ri123 D0ri123 closed this Oct 13, 2024
@agilgur5
Copy link

It looks like this was a near verbatim copy of #13682? If you are copying someone else's code, title, or description, please make sure to cite and credit your source. This makes no mention of #13682 currently, while copying its description entirely yet missing all the formatting (the title and description are therefore worse than the formatted and edited ones as well).

@leehosu was this done with your permission?

@agilgur5 agilgur5 added area/cli The `argo` CLI area/docs Incorrect, missing, or mistakes in docs solution/duplicate This issue or PR is a duplicate of an existing one labels Oct 14, 2024
@agilgur5
Copy link

@leehosu was this done with your permission?

Yes it was per #13706 (comment)

@D0ri123 D0ri123 reopened this Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cli The `argo` CLI area/docs Incorrect, missing, or mistakes in docs solution/duplicate This issue or PR is a duplicate of an existing one
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add examples to commands' --help output
3 participants