Roger Peppe

Results 283 comments of Roger Peppe

This is indeed an real problem. Here's a test case suitable for adding as `cmd/cue/cmd/testdata/script/modget_version_not_present.txtar`: ``` # Check that cue mod get will upgrade an existing # dependency when that...

> So the presence of the `"runs-on"` field is sufficient to know that we are in the `#normalJob` "leg" of the `matchN` (which could equally be a disjunction because both...

> ``` > strategy: value (banana: 5) does not choose a branch in the discriminator #jobStrategy > ``` I think it's interesting to consider that if you'd written `strategy: parallel:...

In the JSON Schema documentation, [it says]( https://json-schema.org/understanding-json-schema/reference/annotations): > The `default` keyword specifies a default value. This value is not used to fill in missing values during the validation process....

There are a couple of issues I'd highlight here. Firstly, the generated reference is just incorrect: a period is not a valid separator in JSON Pointer, and the actual position...

> I'd also add that changing `interface{}` with `any` in the public interfaces could be technically considered a breaking change, so let's not include this for now. Drive-by comment: as...

This is clearly a bug of some sort: in general in CUE, embedding `{X}` should be identical to embedding `X` but here, if we remove the `{}` around `{foo: #Foo}`,...

Here's a testscript reproducer of a slightly more minimal example: ``` exec go mod tidy exec go run . cmp stdout want-stdout -- go.mod -- module test require cuelang.org/go v0.11.0...

> @felixge - thanks for the input. Yours sounds more like an explicit include list situation? ... or perhaps a case where _exactly_ the contents of the directory that are...

Here's a testscript that demonstrates the issue: ``` exec cue cmd test cmp stdout want-stdout -- want-stdout -- {"normal":true} -- cue.mod/module.cue -- module: "cue.example/foo" language: version: "v0.9.2" -- foo_tool.cue --...