Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
    Ross A. Baker
    @rossabaker
    That's the good news.
    The bad news is that it's apparently already broken.
    I'm seeing if I can reproduce.
    That's annoying that the inference is better on 2.12. Most of the time those do about the same thing. 2.13 is where it gets miserable.
    Mostly around collections. Otherwise, even 2.13 is pretty easy.
    Oh, I bet this is a Java8 issue.
    Scala 2.12 lets you use Scala functions where Java functions are needed. 2.11 doesn't give you that out of the box.
    Tim Spence
    @TimWSpence
    Oh yeah, I got the intention of Chris’ change, I was just saying it needs a code change as well. Which you have now done so thank you very much! :) Although it looked like that library also supports .asJava syntax which might look slightly less nasty?
    Ross A. Baker
    @rossabaker
    Oh... yeah, definitely. I was surprised how ugly it was.
    .asJava is more consistent with the more familiar JavaConverters
    Well, or not:
    [error] /home/ross/src/cats-stm/core/src/main/scala/io/github/timwspence/cats/stm/STM.scala:180:44: type mismatch;
[error]  found   : <error> => <error>
[error]  required: ?{def asJava: ?}
[error] Note that implicit conversions are not applicable because they are ambiguous:
[error]  both method enrichAsJavaConsumer in trait Priority1FunctionConverters of type [T](sf: T => Unit)scala.compat.java8.functionConverterImpls.RichFunction1AsConsumer[T]
[error]  and method enrichAsJavaIntFunction in trait Priority1FunctionConverters of type [R](sf: Int => R)scala.compat.java8.functionConverterImpls.RichFunction1AsIntFunction[R]
[error]  are possible conversion functions from <error> => <error> to ?{def asJava: ?}
[error]         entry.tvar.pending.updateAndGet((m => m + (txId -> pending)).asJava)
    Christopher Davenport
    @ChristopherDavenport
    :laughing: Thats an error message.
    So the plan is merge ross’s then enable ci, then integrate mine?
    Tim Spence
    @TimWSpence
    Ouch! That's a shame! Yeah I'll merge Ross's. Surely we need to merge yours to enable CI Chris? Or was that what you meant?
    Christopher Davenport
    @ChristopherDavenport
    Right, once you merge ross’s we can turn on CI and make sure it all goes smoothly.
    Tim Spence
    @TimWSpence
    Cool. I'll be home shortly so I'll merge it then
    Just did it on my phone instead. Thanks Ross!
    I guess I need to enable a build on Travis before we merge yours Chris?
    Christopher Davenport
    @ChristopherDavenport
    I think that would be a good choice.
    Tim Spence
    @TimWSpence
    😂
    I just wanted to check there was nothing else I needed to do
    Christopher Davenport
    @ChristopherDavenport
    Those environmental variables in travis get you set for very quick iteration.
    Then we should get the project on scala-steward so that way you can get automatic updates. I can walk you through the process, or with your permission I can add it.
    Tim Spence
    @TimWSpence
    Cool, I’ll enable and merge now
    What is scala steward?
    Christopher Davenport
    @ChristopherDavenport
    https://github.com/fthomas/scala-steward - A project that will automatically file PR’s to your repository when dependencies update.
    You just need to add TimWSpence/cats-stm to this file
    Tim Spence
    @TimWSpence
    Thanks!
    Just generating the github token
    Is there any way to restrict it to just have access to cats-stm?
    the default seems to be access to all my private repos AFAICT
    Christopher Davenport
    @ChristopherDavenport
    Yeah. Unfortunately thats how the tokens work.
    Tim Spence
    @TimWSpence
    ouch
    Christopher Davenport
    @ChristopherDavenport
    I use one offs that way if I need to replace them I can kill off the token if it leaks.
    Tim Spence
    @TimWSpence
    Yep, called it cats-stm so I can keep track of it :)
    Christopher Davenport
    @ChristopherDavenport
    Note: When this get traction that documentation branch will easily become your largest source of commits.
    Worrisome travis log - here
    Tim Spence
    @TimWSpence
    argh special characters in pw
    Christopher Davenport
    @ChristopherDavenport
    Likely snapshot will fail but site will go through, Unless failure of first after_success blocks the second step
    @rossabaker I’ve never seen this before, have you Link
    @TimWSpence Do you have a gh-pages branch?
    Hmmm, you do.
    Ahh, I see, I was too conservative in what I brough over.
    Tim Spence
    @TimWSpence
    Think I’ve escaped the pw correctly this time
    Do you mean there’s something else that needs brought over @ChristopherDavenport ?
    Christopher Davenport
    @ChristopherDavenport
    Something else. It needs to pull the token from env, That was my fault.
    I knew I had more I needed to pull in but second guessed myself for what you had.
    @TimWSpence Do you mind if we add the Scala CoC to the project as well? Would be nice to ensure a welcoming atmospher.
    Tim Spence
    @TimWSpence
    Not at all!
    I’ve created fthomas/scala-steward#462 btw
    Christopher Davenport
    @ChristopherDavenport
    Woot, I’ll put my template in with your information.