https://kotlinlang.org logo
Channels
Powered by
#arrow-contributors
  • t

    thanh

    10/15/2019, 9:25 AM
    But it is also generated a wrong import 
    import kotlin.Class
  • t

    thanh

    10/15/2019, 9:26 AM
    I'm not sure it's poet issue or our annotation processor issue. And if it's our annotation processor issue, should I try to fix it or just wait for Arrow meta
  • i

    Imran/Malic

    10/15/2019, 9:46 AM
    Without code the import is correct because your using 
    kotlin.Class
    in your function
  • t

    thanh

    10/15/2019, 9:46 AM
    No I'm using 
    java.lang.Class
  • t

    thanh

    10/15/2019, 9:47 AM
    and there is no 
    kotlin.Class
    I think, it's a compiler error.
  • i

    Imran/Malic

    10/15/2019, 9:47 AM
    I cant help you without code
  • t

    thanh

    10/15/2019, 9:48 AM
    should I create a PR?
  • t

    thanh

    10/15/2019, 11:00 AM
    Here is the gist of what I'm doing: https://gist.github.com/lenguyenthanh/a8a01ff5a39e93eeb0a6646fd4e83568. 
    ListKFunctorFilter.kt
    is the generated file.
  • b

    Bob Glamm

    10/17/2019, 2:08 AM
  • b

    Bob Glamm

    10/17/2019, 2:09 AM
    Please let me know if I need to do anything else, but that should fix the SetK.combineAll() issue and includes a regression test to ensure that particular issue doesn't happen again
  • p

    pakoito

    10/17/2019, 8:08 AM
    yay
  • p

    pakoito

    10/17/2019, 8:08 AM
    😄
  • p

    pakoito

    10/17/2019, 8:08 AM
    could you please move the test with the others?
  • p

    pakoito

    10/17/2019, 11:15 AM
    @Bob Glamm and @kitttn have both sent the same PR hahaha: https://github.com/arrow-kt/arrow/pull/1720/files
  • k

    kitttn

    10/17/2019, 11:15 AM
    Damn, this was the channel 😅
  • k

    kitttn

    10/17/2019, 11:16 AM
    Well, Bob was first, so he deserves it 🙂
    ❤️ 2
  • k

    kitttn

    10/17/2019, 11:17 AM
    Just wanted to get some input on including 
    .monoid()
    tests into 
    SetK
    , whether this is the way it should go or i explicitly should declare it somewhere else?
  • p

    pakoito

    10/17/2019, 11:52 AM
    both are correct, I'm not sure if the MonoidLaws exercise 
    plus
    in this case
  • t

    thanh

    10/17/2019, 1:09 PM
    do we have zio for kotlin?
  • a

    aballano

    10/17/2019, 1:11 PM
  • t

    thanh

    10/17/2019, 1:12 PM
    that's nice
  • t

    thanh

    10/17/2019, 3:04 PM
    how difficult it is to create 
    ObservalbleZ<E,A>
    to simulate ZIO stream?
  • p

    pakoito

    10/17/2019, 3:16 PM
    a v1 that does basic stuff, easy-ish
  • p

    pakoito

    10/17/2019, 3:16 PM
    something that's solid, VERY hard
  • t

    thanh

    10/17/2019, 4:25 PM
    how about a wrapper over Observable?
  • p

    pakoito

    10/17/2019, 4:37 PM
    oh that'd wok just fine
  • r

    raulraja

    10/20/2019, 11:49 AM
    @thanh we are building bifunctor IO which works similar to ZIO and a Stream data type for Fx
    👍 2
  • r

    raulraja

    10/20/2019, 11:50 AM
    Let's focus our efforts together there
    💪 2
  • p

    pakoito

    10/20/2019, 3:35 PM
  • p

    pakoito

    10/20/2019, 3:35 PM
    bit of a small panic lol
1...103104105...115Latest