-
Notifications
You must be signed in to change notification settings - Fork 301
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
Type-checking values with disjunct defaults produces incorrect output #3250
Labels
Comments
A default is not "picked" by the |
cueckoo
pushed a commit
that referenced
this issue
Oct 18, 2024
`@if` tags are currently treated as false when they're inside a non-package file inside a module. Add a test case for this, to be fixed in a subsequent CL. For #3250. Signed-off-by: Roger Peppe <[email protected]> Change-Id: I1aafec01ce2586158488bb30d77506622e53eaee Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1202737 Unity-Result: CUE porcuepine <[email protected]> TryBot-Result: CUEcueckoo <[email protected]> Reviewed-by: Daniel Martí <[email protected]>
cueckoo
pushed a commit
that referenced
this issue
Oct 18, 2024
We were treating files directly specified on the command line as outside of the module when actually they should not be. Fixed #3250 Signed-off-by: Roger Peppe <[email protected]> Change-Id: I6b6649aa0e53ed0184a2b3d0648f86bbd98561ed Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1202738 Reviewed-by: Daniel Martí <[email protected]> TryBot-Result: CUEcueckoo <[email protected]> Unity-Result: CUE porcuepine <[email protected]>
cueckoo
pushed a commit
that referenced
this issue
Oct 22, 2024
`@if` tags are currently treated as false when they're inside a non-package file inside a module. Add a test case for this, to be fixed in a subsequent CL. For #3250. Signed-off-by: Roger Peppe <[email protected]> Change-Id: I1aafec01ce2586158488bb30d77506622e53eaee Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1202737 Unity-Result: CUE porcuepine <[email protected]> TryBot-Result: CUEcueckoo <[email protected]> Reviewed-by: Daniel Martí <[email protected]> Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1202872
cueckoo
pushed a commit
that referenced
this issue
Oct 22, 2024
We were treating files directly specified on the command line as outside of the module when actually they should not be. Fixed #3250 Signed-off-by: Roger Peppe <[email protected]> Change-Id: I6b6649aa0e53ed0184a2b3d0648f86bbd98561ed Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1202738 Reviewed-by: Daniel Martí <[email protected]> TryBot-Result: CUEcueckoo <[email protected]> Unity-Result: CUE porcuepine <[email protected]> Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1202873
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What version of CUE are you using (
cue version
)?Does this issue reproduce with the latest stable release?
I haven't yet tested with cue v0.9.2, but nothing in the CHANGELOG indicates that this would be affected.
What did you do?
What did you expect to see?
What did you see instead?
The text was updated successfully, but these errors were encountered: