mappers-go icon indicating copy to clipboard operation
mappers-go copied to clipboard

Update device.go

Open alongL opened this issue 2 years ago • 2 comments

If Desired.Value is empty, don't set the value.

Maybe we should return only the Desired.Type is not string and value is null.

alongL avatar Sep 30 '22 07:09 alongL

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: To complete the pull request process, please assign luogangyi after the PR has been reviewed. You can assign the PR to them by writing /assign @luogangyi in a comment when ready.

The full list of commands accepted by this bot can be found 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

kubeedge-bot avatar Sep 30 '22 07:09 kubeedge-bot

Thanks for the notice. However, the desired value could be empty when someone just care about the value but not setting it.

sailorvii avatar Oct 17 '22 07:10 sailorvii