-
-
Notifications
You must be signed in to change notification settings - Fork 232
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
Standardizing on Cats? #345
Comments
Hi @a1russell, scalajs-react doesn't have a dependency on Scalaz, although it used to a long, long time ago. Now it has an optional extension module that provides Scalaz support but neither Users are free to use the Scalaz module, the Cats module, neither or both. :) Does that address your query? |
Oh! Is it just the tests that depend on it, then? I saw Scalaz in the build file and guess I did not read it thoroughly enough. |
Well it's a bit unconventional but only the tests of the |
Ah, okay, hah. I was excited to see #341 get merged. Will it be ported to the neo branch, as well? |
The neo branch became 1.0.0-RC1 which was merged into master. Master is the
new neo branch now so in that sense, yes; done! In fact it's been released
already too because it was part of 1.0.0-RC2.
|
Oh, I understand now. That's great! Closing this. |
No worries; cheers!
…On 25 March 2017 at 08:42, Adam Russell ***@***.***> wrote:
Oh, I understand now. That's great! Closing this.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#345 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAMYt3sSZ680iVfYrC-j8Mfp4nMtgvYaks5rpDjggaJpZM4MotEk>
.
|
Since 1.0.0 hasn't quite been released yet, I'm wondering whether we can discuss standardizing on Cats rather than Scalaz.
As evidenced by the recently announced joint effort by popular libraries doobie, http4s, and Matryoshka, Cats is gaining momentum versus Scalaz.
In fact, I've personally avoided using scalajs-react in the past in part because of the Scalaz dependency/association.
I'm curious whether you might consider moving to Cats instead.
The text was updated successfully, but these errors were encountered: