Closed alexzielenski closed 11 months ago
I think that's fine, thanks /lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: alexzielenski, apelisse
The full list of commands accepted by this bot can be found here.
The pull request process is described here
You need to fix a gofmt thingy.
New changes are detected. LGTM label has been removed.
@alexzielenski: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
pull-structured-merge-diff-test | f8de3760702eec0ef6fedce0f90340ddb5a6799f | link | true | /test pull-structured-merge-diff-test |
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.
/close
I think we abandoned the idea.
@apelisse: Closed this PR.
mirrors behavior of existing Set for maps
needed to write a
Default
function that applies schema defaults to avalue.Value
in Declarative Validation/assign @apelisse