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 4 months ago — Latest diff uploaded 4 months, 2 weeks 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.

Changes between revision 1 and 2

orig

Commits

	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.	abebbed7cfb4ff23940c84b8b850a81d39c194cb	David Trowbridge
	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

Files

djblets/siteconfig/forms.py

djblets/siteconfig/forms.py