Skip to content

AI spam#466

Closed
wayyoungboy wants to merge 1 commit into
pallets:mainfrom
wayyoungboy:fix-click-840-blueprint-cli-465
Closed

AI spam#466
wayyoungboy wants to merge 1 commit into
pallets:mainfrom
wayyoungboy:fix-click-840-blueprint-cli-465

Conversation

@wayyoungboy

Copy link
Copy Markdown

Fixes #465.\n\nThe blueprint CLI test was passing Ellipsis as cli_group even though the public type is str | None. Click 8.4 validates command names in a way that exposes this invalid test value, so this uses the blueprint's explicit group name instead.\n\nVerification:\n- python -m pytest -o addopts= tests/test_blueprints.py::test_cli_blueprints -q\n- direct Click 8.4.1 blueprint CLI invocation\n- python -m pytest -o addopts= tests/test_blueprints.py -q\n- git diff --check

@davidism davidism closed this Jun 6, 2026
@davidism davidism changed the title Fix blueprint CLI test group for Click 8.4 AI spam Jun 6, 2026
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.

[self-tests] test_cli_blueprints fail with click 8.4.0

2 participants