search

cometkim / rescript-vitest

ReScript bindings to Vitest
MIT License
42 stars 9 forks source link

Not working with Rescript v11 due to uncurried by default #14

Closed mxj4 closed 5 months ago

mxj4 commented 1 year ago

Recently, ReScript v11 beta switched to uncurried by default, and this binding stops working. I tried adding

"uncurried": false,

in bsconfig.json, but the compiler still compiles functions without @uncurry annotation to uncurried functions, not sure if this is a rescript compiler issue, or maybe I modified bsconfig.json incorrectly, or maybe there is some sort of caching ignoring my modification to bsconfig.json deep under node_modules folder.

cometkim commented 1 year ago

IMO, ReScript should ensure backwards compatibility when uncurried: false explicitly. Are there any issues upstream?

cometkim commented 1 year ago

Currying also causes problem with vitest interop, I plan to fix it in the next major version.