You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In particular, to catch that interpreting them as JSON or YAML
currently results in panics.
The following patches will resolve these panics, showing a clear
improvement in the behavior of the added testscript.
For #1790.
For #2714.
Signed-off-by: Daniel Martí <[email protected]>
Change-Id: I91cc75b7ae6eff50cf61c463d0778a41bec3e521
Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1198873
TryBot-Result: CUEcueckoo <[email protected]>
Reviewed-by: Roger Peppe <[email protected]>
What version of CUE are you using (
cue version
)?Also tested on
cue version v0.4.4-0.20220706163258-2795dcc4db87
Does this issue reproduce with the latest release?
Yes.
What did you do?
Attempt to unmarshal YAML content that is just a comment using
encoding/yaml
.Given
test.cue
with:Running
cue eval test.cue
What did you expect to see?
Certainly not a panic. Likely an empty output or bottom?
What did you see instead?
A panic:
The text was updated successfully, but these errors were encountered: