Skip to content

Enforce required flag#1990

Merged
davidism merged 1 commit into
pallets:mainfrom
dannysepler:dsepler/enforce-required-flag
Feb 20, 2022
Merged

Enforce required flag#1990
davidism merged 1 commit into
pallets:mainfrom
dannysepler:dsepler/enforce-required-flag

Conversation

@dannysepler
Copy link
Copy Markdown
Contributor

@dannysepler dannysepler commented Jul 5, 2021

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 dsepler/enforce-required-flag branch from 1c7862f to 4083008 Compare February 20, 2022 19:08
@davidism davidism merged commit 4262661 into pallets:main Feb 20, 2022
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 7, 2022
@dannysepler dannysepler deleted the dsepler/enforce-required-flag branch March 15, 2022 17:36
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.

required=True does not work for flags

2 participants