Enhance typing for KeyValueForm.

Review Request #14437 — Created May 13, 2025 and submitted — Latest diff uploaded

Information

Djblets
release-6.x

Reviewers

This change makes it so KeyValueForm is typed as a generic, allowing us
to specify the type of the instance member which is operated on. The
default implementations of get and set assume a dict-like interface, so
I've additionally added some runtime-checkable protocols for those.

  • Ran unit tests.
  • Verified that type checkers were happy when creating KeyValueForm
    subclasses.

Diff Revision 2

This is not the most recent revision of the diff. The latest diff is revision 4. See what's changed.

orig
1
2
3
4

Commits

First Last Summary ID Author
Enhance typing for KeyValueForm.
This change makes it so KeyValueForm is typed as a generic, allowing us to specify the type of the `instance` member which is operated on. The default implementations of get and set assume a dict-like interface, so I've additionally added some runtime-checkable protocols for those. Testing Done: - Ran unit tests. - Verified that type checkers were happy when creating KeyValueForm subclasses.
97a611feae2ea8969aadad0f0c2ac23b45907d16 David Trowbridge
djblets/extensions/forms.py
djblets/forms/forms/key_value_form.py
djblets/integrations/forms.py
djblets/siteconfig/forms.py
Loading...