Skip to content

cp: fix(cli): accept -v/--verbose before the subcommand; stop suggesting a flag as its own correction (2303) into r0.5.0 - #2335

Merged
kajalj22 merged 2 commits into
r0.5.0from
cherry-pick-2303-r0.5.0
Aug 5, 2026
Merged

cp: fix(cli): accept -v/--verbose before the subcommand; stop suggesting a flag as its own correction (2303) into r0.5.0#2335
kajalj22 merged 2 commits into
r0.5.0from
cherry-pick-2303-r0.5.0

Conversation

@svcnvidia-nemo-ci

Copy link
Copy Markdown

beep boop [🤖]: Hi @e-dobrowolska 👋,

we've cherry picked #2303 into  for you! 🚀

Please review and approve this cherry pick by your convenience!

…a flag as its own correction (#2303)

Related to #1434
Fixes DEFECT A — `-v` / `--verbose` is not global.

Issue: `-v`/`--verbose` was registered only on leaf subparsers, so `gym
-v list benchmarks` failed while the trailing form worked - it is now on
the top-level parser too. The "did you mean?" hint also matched a
misplaced flag against itself; the rejected token is now excluded from
its own candidate set.

Signed-off-by: Ewa Dobrowolska <edobrowolska@nvidia.com>
Co-authored-by: Anwith Kiran <anwithk@nvidia.com>
Signed-off-by: NeMo Bot <nemo-bot@nvidia.com>
@svcnvidia-nemo-ci

Copy link
Copy Markdown
Author

/ok to test 6e2f3ec

@copy-pr-bot

copy-pr-bot Bot commented Aug 4, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@kajalj22

kajalj22 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

/ok to test 69d26fa

@kajalj22
kajalj22 enabled auto-merge (squash) August 5, 2026 02:16
@kajalj22
kajalj22 merged commit 8f4854e into r0.5.0 Aug 5, 2026
29 checks passed
@kajalj22
kajalj22 deleted the cherry-pick-2303-r0.5.0 branch August 5, 2026 02:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants