community icon indicating copy to clipboard operation
community copied to clipboard
verified publisher icon kubernetes

Discourage usage of nullable on API types

Open JoelSpeed opened this issue 7 months ago • 2 comments

From conversation around #8486, we realised that the usage of nullable is problematic and would like to discourage new usage of it.

JoelSpeed avatar Jun 13 '25 15:06 JoelSpeed

Thanks for this clarification!

fabriziopandini avatar Jun 17 '25 15:06 fabriziopandini

(and squash down to a single commit)

liggitt avatar Jul 02 '25 15:07 liggitt

/lgtm /approve

liggitt avatar Jul 02 '25 15:07 liggitt

Had to rebase to fix conflicts after my updates around optional/required serialization merged

JoelSpeed avatar Jul 03 '25 09:07 JoelSpeed

/approve

liggitt avatar Jul 03 '25 14:07 liggitt

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JoelSpeed, liggitt, soltysh

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Sep 18 '25 09:09 k8s-ci-robot

/lgtm

soltysh avatar Sep 18 '25 09:09 soltysh