Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Sep 30 06:57
    TonioGela review_requested #4243
  • Sep 28 19:13
    TonioGela synchronize #4243
  • Sep 28 12:19
    TonioGela synchronize #4243
  • Sep 27 20:49
    TonioGela synchronize #4243
  • Sep 27 16:46

    typelevel-steward[bot] on scala3-library-3.2.0

    Update scala3-library, ... to 3… Regenerate GitHub Actions workf… (compare)

  • Sep 27 16:46
    typelevel-steward[bot] opened #4310
  • Sep 26 13:59
    joroKr21 commented #4302
  • Sep 26 13:09
    armanbilge milestoned #4309
  • Sep 26 13:09
    armanbilge milestoned #4309
  • Sep 26 13:09
    armanbilge labeled #4309
  • Sep 26 12:24
    armanbilge milestoned #4308
  • Sep 26 12:13
    bplommer opened #4309
  • Sep 26 11:07
    SystemFw opened #4308
  • Sep 26 11:07
    djspiewak edited #4308
  • Sep 25 12:26
    TonioGela commented #401
  • Sep 25 12:25
    TonioGela commented #401
  • Sep 23 17:31
    rossabaker commented #4307
  • Sep 23 14:54
    armanbilge closed #4307
  • Sep 23 14:34
    johnynek commented #4307
  • Sep 23 13:53
    SANTAFAMILIA1 opened #4307
Adelbert Chang
@adelbertc
likewise
Ross A. Baker
@rossabaker
I saw cats-contrib for a while in the menu while the header was cats-dev. I got to passively watch the whole metamorphosis.
Kai(luo) Wang
@kailuowang
I didn't expect that either.
Kai(luo) Wang
@kailuowang
@/all last call for weigh in #1682, if the votes remain as is, I think we are probably going to 1) stop publish cats bundle which includes cats-core, cats-free, and cats-laws 2) start to publish cats-tests. alleycats will remain a separate repo.
Miles Sabin
@milessabin
Just voted.
Kai(luo) Wang
@kailuowang
thanks
Yilin Wei
@yilinwei
Test failure in https://travis-ci.org/typelevel/cats/jobs/236421840 is quite concerning
Daniel Spiewak
@djspiewak
@yilinwei lol already restarted that build. :-) I think I know what's causing it though
@kailuowang question re: 1.0.0-MF: are we freezing changes that aren't explicitly slated for 1.0? or are we still merging minor things?
Kai(luo) Wang
@kailuowang
IMO, we can certainly merge minor things. let’s tag them as 1.0.0-MF as we go.
Daniel Spiewak
@djspiewak
ok
Yilin Wei
@yilinwei
Thanks @djspiewak
Daniel Spiewak
@djspiewak
:-)
Yilin Wei
@yilinwei
Am I right that the Eq isn't working as intended?
It was a vague guess from looking at the discipline test
Daniel Spiewak
@djspiewak
yeah, Eq for Function1 on javascript is flaky
I think it was @kailuowang who found that (at least, that's what my vague memory thinks)
Yilin Wei
@yilinwei
How come, don't they both essentially pump Arb[A] into the function and compare the results?
Daniel Spiewak
@djspiewak
they do
I don't remember the exact details of why it's flaky, but it comes down to the fact that generating arbitrary values is really slow on JS
so it can only pump a few values on javascript
(I think it's like, 5?)
Yilin Wei
@yilinwei
The failure wasn't that it was hanging though; the assertion was incorrect.
Daniel Spiewak
@djspiewak
hmm
Yilin Wei
@yilinwei
It should boil down to None == None after the function has been invoked.
Edmund Noble
@edmundnoble
Eq for function1 is flaky on all platforms.
JS just happens to have few testing samples.
Function1's Eq gives you false positives. And then when your test negates the result, you get a false negative.
Kai(luo) Wang
@kailuowang
we should probably bump up the numbers of samples in js now that we have matrix build and js is no longer the bottle neck
Yilin Wei
@yilinwei
Ah, I understand.
Kai(luo) Wang
@kailuowang
crap, I click the wrong button and commited typelevel/cats@80d56e6 to master without PR
I am going to revert it back.
Kai(luo) Wang
@kailuowang
just submitted a PR #1705 to revert it, but if you thumbs up the original change , we might not need to merge the revert PR.
I apologize. Looks like that less sleep is taking a toll on me.
Edmund Noble
@edmundnoble
No worries. Where's the original change?
Kai(luo) Wang
@kailuowang
typelevel/cats@80d56e6
Edmund Noble
@edmundnoble
For the record, :+1: from me.
Kai(luo) Wang
@kailuowang
would you post it to the PR and indicate if the thumb is for the original change or the revert. Thanks!
Edmund Noble
@edmundnoble
Kk I'll put it there
Yilin Wei
@yilinwei
I think #1659 looks fine; codecov is down, though not really sure how to read it because I've tested all the new code and it was pretty much moving the code from one place to another.
Yilin Wei
@yilinwei
#1332 can be closed?
Kai(luo) Wang
@kailuowang
@yilinwei I will get back to you on those two issues later if no one else beats me to it.
Yilin Wei
@yilinwei
Apologies; since I don't follow all the discussion all the time, the issues are the only way I really know what's going on and pick up new issues.
Kai(luo) Wang
@kailuowang
No problem. I had a hard time following discussions as well, mostly because Edmund couldn't stop talking. :smile: . Thankfully cats-dev seems a lot easier to follow
Edmund Noble
@edmundnoble
<3
Kai(luo) Wang
@kailuowang
any maintainer has 5 minutes to spare? #1709 should be a quick review.
Daniel Spiewak
@djspiewak
@kailuowang done
Kai(luo) Wang
@kailuowang
that’s quick. Thanks!
also Traverse1 needs a better name in #1611.