Skip to content

Move processing of help from @option into Option#1987

Merged
davidism merged 2 commits into
pallets:mainfrom
janluke:fix/option-help-processing
Feb 20, 2022
Merged

Move processing of help from @option into Option#1987
davidism merged 2 commits into
pallets:mainfrom
janluke:fix/option-help-processing

Conversation

@janluke

@janluke janluke commented Jul 4, 2021

Copy link
Copy Markdown
Collaborator

Checklist:

  • Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
  • Add or update relevant docs, in the docs folder and in code.
  • Add an entry in CHANGES.rst summarizing the change and linking to the issue.
  • Add .. versionchanged:: entries in any relevant code docs.
  • Run pre-commit hooks and fix any issues.
  • Run pytest and tox, no tests failed.

@davidism davidism added this to the 8.1.0 milestone Aug 14, 2021
@davidism
davidism force-pushed the fix/option-help-processing branch from 40bb088 to 89c1b9a Compare February 20, 2022 18:41
@davidism
davidism force-pushed the fix/option-help-processing branch from 89c1b9a to 721c8b0 Compare February 20, 2022 18:42
@davidism
davidism merged commit 22d3886 into pallets:main Feb 20, 2022
@janluke
janluke deleted the fix/option-help-processing branch February 20, 2022 20:01
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@option processing help argument despite docstring saying that it forwards arguments unchanged

2 participants