Hopefully, a self explanatory issue title and patch :)

CommentFileSizeAuthor
field-interface.patch12.64 KBeffulgentsia

Comments

Status: Needs review » Needs work

The last submitted patch, field-interface.patch, failed testing.

effulgentsia’s picture

effulgentsia commented
Status: Needs work » Needs review

field-interface.patch queued for re-testing.

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Status: Needs review » Reviewed & tested by the community

Straight forward cleanup

yched’s picture

yched commented

Yup, RTBC+1

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed
Issue tags: +Approved API change

This represents an API change, but it's consistent with other places we're typehinting interfaces rather than classes, so I believe this makes sense.

Committed and pushed to 8.x. Thanks.

Automatically closed -- issue fixed for 2 weeks with no activity.