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

ES2015 is not correct #11

Closed
jussi-kalliokoski opened this issue Jul 28, 2015 · 4 comments
Closed

ES2015 is not correct #11

jussi-kalliokoski opened this issue Jul 28, 2015 · 4 comments

Comments

@jussi-kalliokoski
Copy link

The bind syntax is not part of ES2015 and might not make it to ES2016 either, I'd maybe change the reference in 4ba7036#diff-04c6e90faac2675aa89e2176d2eec7d8R32 to something like "future JS feature in proposal stage". :)

Nice, btw, saves me from adding a curry to Trine! Unless I want curry to do something different. :D That said, I personally find it a bit weird that it's mixing partial application and currying, but that's just me. :P

@stoeffel
Copy link
Collaborator

The bind syntax is not part of ES2015 and might not make it to ES2016 either, I'd maybe change the reference in 4ba7036#diff-04c6e90faac2675aa89e2176d2eec7d8R32 to something like "future JS feature in proposal stage". :)

Thanks, for the hint. Will fix that.

Nice, btw, saves me from adding a curry to Trine!

😄 @tomekwi and I are working to make the api even simpler. With the next version we will also rename the project to curry-this. As soon as it's ready it would be cool to get your feedback.
I think that we should also link to trine and mention that the two work great together.

@tomek-he-him
Copy link
Member

@jussi-kalliokoski it’s an honor seeing you around here!

I personally find it a bit weird that it's mixing partial application and currying

The reason we’re mixing currying and partial application is simplicity of the API. In a world where the arity of a function is implicit and there many confilicting community conventions you end up with crazy complicated APIs for both things.

And since both currying and partial application are in fact two means to achieve a similar goal, we decided to marry them into one simple API. Have you seen the the draft at the v3 branch?

tomek-he-him pushed a commit that referenced this issue Jul 29, 2015
It’s not ES6 ( #11 )
@jussi-kalliokoski
Copy link
Author

@stoeffel

As soon as it's ready it would be cool to get your feedback.

Will be happy to provide!

I think that we should also link to trine and mention that the two work great together.

Yup! Once you release I'll also add a new section to Trine about libraries that play well together with it. :)

@tomekwi

it’s an honor seeing you around here!

The honor is all yours (meaning the both of you) for creating this. ^.^

The reason we’re mixing currying and partial application is simplicity of the API.
...
And since both currying and partial application are in fact two means to achieve a similar goal, we decided to marry them into one simple API.

That makes sense - have been thinking the same myself a couple of times. I think it's just the old API that confuses me. :) Keep up the good work!

@stoeffel stoeffel closed this as completed Aug 5, 2015
@stoeffel
Copy link
Collaborator

stoeffel commented Aug 9, 2015

@jussi-kalliokoski fyi. we did release v3. thx for your input.

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

No branches or pull requests

3 participants