Skip to content
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

Add a Gitter chat badge to README.md #1

Closed
wants to merge 1 commit into from

Conversation

gitter-badger
Copy link

non/cats now has a Chat Room on Gitter

@non has just created a chat room. You can visit it here: https://gitter.im/non/cats.

This pull-request adds this badge to your README.md:

Gitter

Happy chatting.

PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

@non non closed this Jan 28, 2015
mpilquist added a commit that referenced this pull request Feb 1, 2015
Applies another scaladoc hotfix to macro paradise
stew added a commit that referenced this pull request Feb 8, 2015
Add tut to validate SBT command alias;

Yeah, that's good.
non added a commit that referenced this pull request May 11, 2015
1. Remove the "unsafe" Reducible class
2. Rewrite Reducible in terms of reduceLeftTo and reduceRightTo
3. Improve the documentation a bit

There was no need in Cats itself for #1. It seemed better to not
provide a dangerous type unless we had an active need. As far as
.toNEL, which @ceedubs mentioned.

As far as #3, the docs can always be better! ;)
adelbertc added a commit that referenced this pull request Aug 14, 2015
non added a commit that referenced this pull request Aug 24, 2015
This commit makes several changes:

 1. Makes StreamingT ADT nodes final and private[cats].
 2. Provide more type class instances for StreamingT (including Eq)
 3. Introduce a few new methods
 4. Improve documentation substantially
 5. Some minor performance improvements
 6. Fix minor syntax problem.

The most significant change is #1. If you expose the ADT and allow
pattern-matching, then referential transparency requires that operations such
as fa.flatMap(a => f(a).flatMap(g)) and fa.flatMap(f).flatMap(g) produce
identical ADTs. In the case of StreamT this was not possible without
dramatically bloating the number of allocations per "step" in all cases.
By hiding the ADT and preventing pattern-matching (and folds over the
underlying ADT structure) we can ensure that these are equivalent.

The introduction of .zipMap and .izipMap are intended to make the process of
combining streams via pairwise function application a bit less painful. Using
these methods we can define Eq, Order, and so on without creating intermediate
tuples, converting to a new data structure, etc.

The minor syntax problem was a failure of OrderSyntax to extend
PartialOrderSyntax; you can see similar problems anywhere there is an
inheritance chain. Basically, the Ops class for a type class should only
support methods directly added by that class. In other words, even if you are
working with a Group[A], your |+| method should always come from SemigroupOps.
This prevents ambiguities, and ensures that things work properly.

This commit also fixes some minor typos, indentation problems, and removes some
unused methods which were probably not necessary.
non pushed a commit that referenced this pull request Oct 4, 2015
Add WriterT law-checking and tests
@stew stew mentioned this pull request Jan 30, 2016
ceedubs pushed a commit that referenced this pull request Feb 3, 2016
Use sbt-doctest for some Streaming(T) examples
adelbertc pushed a commit that referenced this pull request May 13, 2016
Use instance hierarchy for WriterT group instances
ceedubs pushed a commit that referenced this pull request Jun 11, 2016
Add a short-circuiting implementation of foldLeftM
travisbrown referenced this pull request in travisbrown/cats Aug 20, 2016
@kailuowang kailuowang mentioned this pull request May 25, 2017
26 tasks
ceedubs pushed a commit that referenced this pull request Sep 12, 2017
Refactor scalafix migration rewrite
kailuowang pushed a commit to kailuowang/cats that referenced this pull request Dec 5, 2017
make Ior handleErrorWith to treat Both as valid
@rossabaker rossabaker mentioned this pull request Dec 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants