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

Add redundant check for --yes and --no flags conflict #221

Merged

marcospb19 merged 1 commit into master from flags-no-yes-redundancy-check

Dec 8, 2021

Member

marcospb19 commented Dec 8, 2021

Should be covered by clap, but we're double checking it now


          Add redundant check for --yes and --no flags conflict

775e92c

Should be covered by clap, but we're double checking it now

marcospb19 merged commit b509878 into master

marcospb19 commented

View reviewed changes

src/cli.rs

+                          (false, false) => QuestionPolicy::Ask,
+                          (true, false) => QuestionPolicy::AlwaysYes,
+                          (false, true) => QuestionPolicy::AlwaysNo,
+                          (true, true) => unreachable!(),

Member Author

marcospb19 Dec 8, 2021

Now we'll panic in case it fails.

marcospb19 deleted the flags-no-yes-redundancy-check branch

June 8, 2022 20:51

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

Labels

None yet