Skip to content
This repository has been archived by the owner on Nov 18, 2021. It is now read-only.

doc: tutorial types errors, float not recognized #50

Closed
ghchinoy opened this issue May 24, 2019 · 3 comments
Closed

doc: tutorial types errors, float not recognized #50

ghchinoy opened this issue May 24, 2019 · 3 comments

Comments

@ghchinoy
Copy link

Using types.cue from https://github.com/cuelang/cue/blob/master/doc/tutorial/basics/types.md errors on running cue eval.

$ cue eval -i types.cue 
--- .
unsupported op &((float)*, int):
    ./types.cue:2:8
    ./types.cue:7:10
    
// /home/ghchinoy/dev/cueexperments
{
    point: {
        x:  float
        y:  float
    }
    xaxis: {
        x:  _|_ // unsupported op &((float)*, int)
        y:  float
    }
    yaxis: {
        x:  float
        y:  _|_ // unsupported op &((float)*, int)
    }
    origin: {
        x:  _|_ // unsupported op &((float)*, int)
        y:  _|_ // unsupported op &((float)*, int)
    }
}```
@mpvl
Copy link
Contributor

mpvl commented May 31, 2019

Ah, there was a recent change to make integer literals not floats. I’ll update the example to use number, the now recommended type to use.

I’m close to finishing some tests for the tutorials to allow catching these kind of things during development.

@ghchinoy
Copy link
Author

Thanks @mpvl - please let me know if there's anything I can do to help!

mpvl added a commit that referenced this issue Jun 7, 2019
doesn't fully work yet because commands and
flags are still global.

Issue #50

Change-Id: I281684d0a9bfc72c9c852cf17f1552587e127efb
Reviewed-on: https://cue-review.googlesource.com/c/cue/+/2160
Reviewed-by: Marcel van Lohuizen <mpvl@google.com>
mpvl added a commit that referenced this issue Jun 7, 2019
This to improve testing abilities.

Issue #50

Change-Id: Ib9f5c0cfe4e0885e0189c1e1393bdb6968c384b8
Reviewed-on: https://cue-review.googlesource.com/c/cue/+/2161
Reviewed-by: Marcel van Lohuizen <mpvl@google.com>
mpvl added a commit that referenced this issue Jun 7, 2019
Issue #50

Change-Id: I06be2f1e4dc4ce1d531ddb9c35eb3c11c5693f8d
Reviewed-on: https://cue-review.googlesource.com/c/cue/+/2162
Reviewed-by: Marcel van Lohuizen <mpvl@google.com>
mpvl added a commit that referenced this issue Jun 7, 2019
Issue #50.

Change-Id: I742002d9b3264789d8cbbbe2954859a14a422541
Reviewed-on: https://cue-review.googlesource.com/c/cue/+/2166
Reviewed-by: Marcel van Lohuizen <mpvl@google.com>
@mpvl mpvl closed this as completed in 00c373d Jun 7, 2019
@cueckoo
Copy link

cueckoo commented Jul 3, 2021

This issue has been migrated to cue-lang/cue#50.

For more details about CUE's migration to a new home, please see cue-lang/cue#1078.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants