search

bevry / cson

CoffeeScript-Object-Notation. Same as JSON but for CoffeeScript objects.
Other
1.34k stars 55 forks source link

typechecker.js throw error on require CSON #64

Closed GrimDev closed 9 years ago

GrimDev commented 9 years ago

Hi.

I use cson 3.1.0 and the following error is thrown on "require "cson".

/node_modules/cson/node_modules/extract-opts/node_modules/typechecker/es6/lib/typechecker.js:29
for ( let type of types ) {
            ^^^^^

But the library works properly after that.

Do you know what does it mean ?

balupton commented 9 years ago

Which version of node are you using?

GrimDev commented 9 years ago

I was using nodev0.10.29 of 2014-05 O_o.

It was on a local repository never updated.

No problems now on 4.0.0 :)

Thx

kvz commented 9 years ago

Running into the same thing here on v0.10.24. Is a newer node required for cson?

If so, would it make sense to reflect that in: https://github.com/bevry/cson/blob/master/package.json#L67?

balupton commented 9 years ago

It's just a warning, it can be ignored. Node 0.12 and up won't have this warning.

The latest https://github.com/bevry/esnextguardian has a workaround for this, but hasn't been pushed up to all the other applicable bevry projects yet.

kvz commented 9 years ago

It turns out there was another issue that caused a fatality over here. The warning made it appear it originated from typechecker, tracing back to cson. All good here now.