Channels
#arrow-contributors
- p
pakoito
12/23/2018, 1:07 PMyeah, we delegate everything to gradle, and I test mostly on the CLI🔝 2 - s
simon.vergauwen
12/23/2018, 5:18 PM - j
jacob
12/23/2018, 10:40 PM@pakoito cleaned up https://github.com/arrow-kt/arrow/pull/1219 I think it's ready to go - p
pakoito
12/23/2018, 11:53 PMdone and merged! - j
jacob
12/23/2018, 11:59 PM😄 - j
jacob
12/24/2018, 12:01 AMI think that just leaves boolean and eval. did you still want to talk about what to do with eval? - p
pakoito
12/24/2018, 12:10 AMeval has different permutations depending on the node, right? - p
pakoito
12/24/2018, 12:10 AMor what was the issue? - j
jacob
12/24/2018, 12:31 AMyeah, I think that was the issue. how to combine different eval<A> - j
jacob
12/24/2018, 12:32 AM@pakoito just go with https://github.com/typelevel/cats/blob/master/core/src/main/scala/cats/Eval.scala#L463 ? - p
pakoito
12/24/2018, 12:34 AMI take that coerces everything to now? - p
pakoito
12/24/2018, 12:35 AMooooh - p
pakoito
12/24/2018, 12:35 AMwait, no, I see - p
pakoito
12/24/2018, 12:36 AMea.flatMap{ a -> eb.map { b -> a + b } } - p
pakoito
12/24/2018, 12:36 AMthat deals with lazy and memoized etc just right - j
jacob
12/24/2018, 12:36 AMI think so. returns you an hkt, so you can deal with it however you need to based on the hkt - p
pakoito
12/24/2018, 12:37 AM👍🏼 - p
pakoito
12/24/2018, 12:37 AMadd some tests JiC - p
pakoito
12/24/2018, 12:37 AMthat definition looks right to me - p
pakoito
12/24/2018, 12:37 AMI’m pretty sure that exists somewhere in Monad - j
jacob
12/24/2018, 12:38 AMyes, definitely. will try it with a few different types - p
pakoito
12/24/2018, 12:38 AMI’d actually add it to Monad - p
pakoito
12/24/2018, 12:38 AMseems valuable - p
pakoito
12/24/2018, 12:38 AM…applicative? - p
pakoito
12/24/2018, 12:38 AM…monad? - p
pakoito
12/24/2018, 12:38 AMhmmmm - p
pakoito
12/24/2018, 12:40 AMit’s an specialization ofApplicative#map2 - p
pakoito
12/24/2018, 12:40 AMmeh - p
pakoito
12/24/2018, 12:40 AMnah - j
jacob
12/24/2018, 12:40 AMit's just lift2 combine right?