Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

evaluator: required fields are dropped by cue eval #2305

Closed
rogpeppe opened this issue Mar 22, 2023 · 0 comments
Closed

evaluator: required fields are dropped by cue eval #2305

rogpeppe opened this issue Mar 22, 2023 · 0 comments
Assignees
Labels

Comments

@rogpeppe
Copy link
Member

What version of CUE are you using (cue version)?

0b681f5

Does this issue reproduce with the latest stable release?

N/A

What did you do?

exec cue eval x.cue
cmp stdout want.cue
-- x.cue --
a: x!: 5
-- want.cue --
a: x!: 5

What did you expect to see?

Passing test.

What did you see instead?

> exec cue eval x.cue
[stdout]
a: {}
> cmp stdout want.cue
--- stdout
+++ want.cue
@@ -1,1 +0,0 @@
-a: {}
@@ -0,0 +1,1 @@
+a: x!: 5

FAIL: /tmp/testscript4012701096/x.txtar/script.txtar:2: stdout and want.cue differ
@rogpeppe rogpeppe added NeedsInvestigation Triage Requires triage/attention labels Mar 22, 2023
@rogpeppe rogpeppe added this to the v0.6.0 required fields milestone Mar 22, 2023
@rogpeppe rogpeppe added NeedsFix and removed NeedsInvestigation Triage Requires triage/attention labels Mar 22, 2023
cueckoo pushed a commit that referenced this issue Apr 5, 2023
All information is now present and directly
accessible in Vertex.

Issue #2305

Signed-off-by: Marcel van Lohuizen <mpvl@gmail.com>
Change-Id: I76f6e0e11c8013bbae904a8d1d068f1ffe3a67a8
Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/552168
Unity-Result: CUEcueckoo <cueckoo@cuelang.org>
TryBot-Result: CUEcueckoo <cueckoo@cuelang.org>
Reviewed-by: Daniel Martí <mvdan@mvdan.cc>
Reviewed-by: Roger Peppe <rogpeppe@gmail.com>
@cueckoo cueckoo closed this as completed in d5dd9ec Apr 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants