This change has been marked as completed.

Describe the completed change (optional):

Pushed to release-6.x (90d53c8)

Summary

Enhance typing for KeyValueForm.

Review Request #14437 — Created May 13, 2025 and submitted 1 month, 3 weeks ago — Latest diff uploaded 2 months, 1 week ago

Information

Owner

david*

Repository

Djblets

Branch

release-6.x

Bugs

Depends On

Reviewers

Groups

djblets

People

Description*

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.

Diff Revision 4 (Latest)

orig

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.	5198a036e8c4a39c7dc0e9ff6d5917f15106f6a1	David Trowbridge

Files

	djblets/extensions/forms.py
	djblets/forms/forms/key_value_form.py
	djblets/integrations/forms.py
	djblets/siteconfig/forms.py

djblets/extensions/forms.py

djblets/forms/forms/key_value_form.py

djblets/integrations/forms.py

djblets/siteconfig/forms.py