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.

Sign up for GitHub

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

Jump to bottom

test(help): Remove redundant test #4864

Merged

epage merged 1 commit into clap-rs:master from mattmadeofpasta:remove-redundant-test

Apr 27, 2023

Contributor

mattmadeofpasta commented Apr 27, 2023

The tests positional_display_val_names_required and positional_display_val_names_req are equivalent.

positional_display_val_names_required was added in Aug 2022 in 02db304#diff-f1b965dbd6b694e7419437ace420b4fac5e8e0366848860ce5e63523fd7b835fR4430-L4438.

Keep positional_display_val_names_required and remove positional_display_val_names_req.


          test(help): Remove redundant test

296fec9

epage merged commit 6b23edc into clap-rs:master

21 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment