Daniel Martí

Results 1731 comments of Daniel Martí

Does Docker merge the configs, though? Because we should ideally be mimicking whatever behavior Docker (or Podman) has. Doing something entirely different could lead to very confusing behavior for end...

@bogdanbojan that looks about right. Can you send a patch via Gerrit? See our contributng doc for pointers. Once merged, we'd need a small follow-up to bump the dependency and...

no worries, the help is welcome! the OCI repo patch is merged; the remaining work is bumping the dependency here, and probably adding a tiny integration test to `cmd/cue/cmd/testdata/script/registry_auth.txtar` too.

Thanks! Merged. I've subscribed to https://github.com/docker/cli/issues/6156 for now, so if upstream decides to tweak the priority logic between the two env vars, we can adapt accordingly.

For point 2 above, it's important to note that the guide's `go mod init` command has to be run with the latest Go version for the go/packages error above to...

The backport release happened here: https://github.com/cue-lang/cue/releases/tag/v0.12.1 I've retitled the issue to be about the cuelang.org sanity checking that Paul mentions above.

I sort of agree with your point, but I don't think there is one true standard that we can enforce here. If I name your four original formatting examples above...

I don't agree with that. gofumpt doesn't exist to invent and enforce new standards of formatting. Forms A and C above are currently widespread, and nothing is wrong with either...

Could you please explain why you expect `propABase` to be a string type rather than a reference? In the original CUE it does use a reference to `propAParent`, so the...

Ah I see, thanks for the clarification. I'll let @rogpeppe chime in.