Skip to content

Fix and refactor make_default_short_help (issue #1849)#1850

Merged
davidism merged 1 commit into
pallets:masterfrom
janluke:fix/make-default-short-help
Apr 14, 2021
Merged

Fix and refactor make_default_short_help (issue #1849)#1850
davidism merged 1 commit into
pallets:masterfrom
janluke:fix/make-default-short-help

Conversation

@janluke
Copy link
Copy Markdown
Collaborator

@janluke janluke commented Apr 14, 2021

Fixes #1849.

I found the old code a little bit difficult to follow, so I rewrote it. As it written now, the code can also be easily generalized to an arbitrary overflow placeholder.

Checklist:

  • Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
  • [Not relevant] 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.
  • [Not relevant] Add .. versionchanged:: entries in any relevant code docs.
  • Run pre-commit hooks and fix any issues.
  • Run pytest and tox, no tests failed.

@janluke janluke force-pushed the fix/make-default-short-help branch from e9d1f4e to e2458ae Compare April 14, 2021 15:59
@davidism davidism force-pushed the fix/make-default-short-help branch from e2458ae to d48b59b Compare April 14, 2021 19:11
@davidism davidism merged commit 82a83fd into pallets:master Apr 14, 2021
@janluke janluke deleted the fix/make-default-short-help branch April 14, 2021 23:37
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 29, 2021
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.

Bug: make_default_short_help can return strings as long as max_length + 3

2 participants