Skip to content

Conversation

@briancavalier
Copy link
Owner

⚠️ BREAKING CHANGE ⚠️

This reverses the "argument" order for ap to be compliant with the same reversal in Fantasy Land 1.0. This will force a 2.0.0 release.

BREAKING CHANGE: This changes the "argument" order for ap(), in
accordance with the same breaking change to the FL spec 1.0.
@coveralls
Copy link

coveralls commented Oct 23, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 99912a4 on update-ap-fl-1.0 into a006e6e on master.

@bergus
Copy link
Contributor

bergus commented Oct 24, 2016

I have to admit that I liked ap much better the way it was before.
Maybe we update to FL2.0 right away, and offer only the prefixed method with the inverted parameter order?

@briancavalier
Copy link
Owner Author

I have to admit that I liked ap much better the way it was before.

Same here 👍

Maybe we update to FL2.0 right away, and offer only the prefixed method with the inverted parameter order?

Ah, that's an interesting possibility! Can you think of any potential repercussions on usability, like users assuming the non-prefixed ap will work like the FL spec?

@evilsoft
Copy link

evilsoft commented Oct 26, 2016

Just a heads up about dem prefixes.

They were required in FL1.0, when the change to ap ( 😢 ) was introduced. This will become important in the community if PRs like this one happen in all other FL interop libs

Notice that it will not call the none prefixed version. This will create havoc for anyone that is using Ramda for point-free madness on any algebra in creed (or any other FL lib for that matter).

And according to the comments on that PR, looks like all but one are in favor of cutting the cord.

@evilsoft
Copy link

I have to admit that I liked ap much better the way it was before.

Amen to that!!! 👍

@briancavalier
Copy link
Owner Author

Hey @evilsoft. Thanks for the additional info.

Based on all of that, I'm feeling very much in favor of @bergus' idea of keeping .ap as it was before this PR, and simply adding all of the namespaced FL 2.0 methods that creed supports (they can simply delegate to the non-namespaces versions). That could be released as a minor version bump. If the .ap vs ['fantasy-land/ap'] difference ends up being confusing for people, we can discuss how to handle it at that point.

How does that sound?

@briancavalier
Copy link
Owner Author

Closing in favor of #86

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.

4 participants