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

Fix for #1031: "multiple not valid with is_flag" from click 8.1.3 #1033

Merged

Conversation

dairiki
Copy link
Contributor

@dairiki dairiki commented May 2, 2022

The latest release of click raises TypeError for Options that are
declared with both multiple and is_flag set. The exception message
suggests using count instead, but that does not do what we want.

This commit rejiggers things to avoid the use of multiple on
is_flag options.

Issue(s) Resolved

Fixes #1031

Related Issues / Links

Description of Changes

Add tests for the `--name`, `--project-file`, `--tree`, and
`--output-path` options to the `lektor project-info` subcommand.
The latest release of click raises `TypeError` for Options that are
declared with both `multiple` and `is_flag` set. The exception message
suggests using `count` instead, but that does not do what we want.

This commit rejiggers things to avoid the use of `multiple` on
`is_flag` options.

Reference: lektor#1031.
@dairiki dairiki force-pushed the bug.1031-click-multiple-not-valid-with-is_flag branch from 99b94a7 to 6238e13 Compare May 2, 2022 21:19
dairiki added a commit that referenced this pull request May 2, 2022
webknjaz added a commit to Omikron-Voiceover/omikron.com.ua that referenced this pull request May 2, 2022
@dairiki dairiki changed the title Fix for #1031: "multiple not valid with is flag" from click 8.1.3 Fix for #1031: "multiple not valid with is_flag" from click 8.1.3 May 2, 2022
@dairiki dairiki merged commit 1431801 into lektor:master May 2, 2022
@dairiki dairiki deleted the bug.1031-click-multiple-not-valid-with-is_flag branch May 2, 2022 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TypeError: 'multiple' is not valid with 'is_flag', use 'count'.
1 participant