-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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! ;)
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
Closed
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
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
Draft
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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:
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.