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

Renamed curry->partial to better match semantics #1979

Merged
merged 2 commits into from Mar 10, 2014
Merged

Renamed curry->partial to better match semantics #1979

merged 2 commits into from Mar 10, 2014

Conversation

msoucy
Copy link
Contributor

@msoucy msoucy commented Mar 3, 2014

Documentation was updated to match, and a deprecated alias was inserted for compatibility.

This sets up the library in preparation for Issue 4391, which would provide a proper "curry" template that follows currying semantics properly.

Documentation was updated to match, and a deprecated alias was inserted
for compatibility.
@Poita
Copy link
Contributor

Poita commented Mar 3, 2014

You should include a deprecation message:

deprecated("Please use std.functional.partial instead.") alias curry = partial;

@msoucy
Copy link
Contributor Author

msoucy commented Mar 3, 2014

Good call - I thought I had done that, it turns out that didn't get committed. Thank you.

@Poita
Copy link
Contributor

Poita commented Mar 3, 2014

LGTM

@andralex
Copy link
Member

sigh

andralex added a commit that referenced this pull request Mar 10, 2014
Renamed curry->partial to better match semantics
@andralex andralex merged commit 35c7439 into dlang:master Mar 10, 2014
@msoucy
Copy link
Contributor Author

msoucy commented Mar 10, 2014

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants