Skip to content
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

Uncurries internals for rescript 11 compatibility #160

Conversation

illusionalsagacity
Copy link
Contributor

@illusionalsagacity illusionalsagacity commented Oct 20, 2024

Changes the internal usage of most externals or function definitions used at runtime to be explicitly uncurried functions. The intent here is that the Js_ modules are changed, not the public API. This will eliminate any ambiguous function calling conventions when the consumer of this library is on ReScript v10 or v11 on curried mode.

@illusionalsagacity illusionalsagacity force-pushed the uncurry-internals-works-on-rescript-10 branch from fb07b98 to d1d4967 Compare October 20, 2024 18:24
@illusionalsagacity illusionalsagacity marked this pull request as ready for review October 20, 2024 18:33
@jeddeloh jeddeloh merged commit ea2f9b7 into jeddeloh:master Oct 24, 2024
1 check passed
@illusionalsagacity illusionalsagacity deleted the uncurry-internals-works-on-rescript-10 branch October 24, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants