Closed dhaiducek closed 3 years ago
@dhaiducek this looks good. Could you please rebase on the latest main
branch and add a unit test for this?
Yes! I'll work on a unit test to cover the other PR also.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dhaiducek, mprahl
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Rather than running an initial validation on the
placementBindingDefaults.name
, this moves the logic deeper, leveraging the new placement flows to determine whether a default binding name is strictly necessary.