Bug: rescript format -all with uncurried false breaks uncurried functions ([email protected])

[DZakh]

It's not critical in this place, but I don't think that formatter should cause changes in the generated code.

[DZakh]

It might be a problem for cases like this.

I'll rewrite it to use @scope instead, but still.

[zth]

Can you check if rc2 fixes it? I committed a potential fix for this that hasn't been released yet: https://github.com/rescript-lang/rescript-compiler/blob/master/CHANGELOG.md#bug-bug-fix

[DZakh]

Yes, it works there. @zth @cknitt I'd really appreciate a release. I've started migrating FE at work to ReScript@11, and this actually happened to be blocking.
Also, here's a fix for failing tests in master #6370

[cknitt]

@DZakh https://github.com/rescript-lang/rescript-compiler/releases/tag/v11.0.0-rc.2 🙂

[DZakh]

Thanks

[DZakh]

Sorry. I've tested it incorrectly. The issue still exists.

[zth]

Can you create a reproduction?

[DZakh]

Yep #6378

[DZakh]

I've added a fix in the PR #6378 after @cristianoc's pointing.