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

Tuple Apply syntax fixes #1363

Closed
3 tasks
adelbertc opened this issue Sep 7, 2016 · 5 comments
Closed
3 tasks

Tuple Apply syntax fixes #1363

adelbertc opened this issue Sep 7, 2016 · 5 comments

Comments

@adelbertc
Copy link
Contributor

Referring to this syntax:

import cats.implicits._

(Option(2), Option(3)).map2(_ + _)
  • No Unapply
  • Should this be under cats.syntax.tuple as it currently is or cats.syntax.apply?
  • Deprecate/remove CartesianBuilder syntax
@non
Copy link
Contributor

non commented Sep 7, 2016

For (1), let's add it!
For (2) I think I'd prefer cats.syntax.apply but I don't feel super strongly if someone else disagrees.
For (3), yes.

@longcao
Copy link
Contributor

longcao commented Nov 17, 2016

As noted by @non, mapN would be a nice name for this: https://gitter.im/typelevel/cats?at=582dd60269eb65323c45819c

@mpilquist
Copy link
Member

👍 for mapN so that folks don't need to manually count the number of values.

@julienrf
Copy link
Contributor

julienrf commented Dec 6, 2016

for Unapply support I remember having tried without success a few month ago. You can see my code here.

@mpilquist
Copy link
Member

@julienrf I think we can drop Unapply support now that we have SI-2712 fixed.

DavidGregory084 added a commit to DavidGregory084/cats that referenced this issue Dec 8, 2016
DavidGregory084 added a commit to DavidGregory084/cats that referenced this issue May 20, 2017
DavidGregory084 added a commit to DavidGregory084/cats that referenced this issue May 20, 2017
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

5 participants