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 zipWith #1883

Closed
wants to merge 35 commits into from
Closed

Add zipWith #1883

wants to merge 35 commits into from

Conversation

LukaJCB
Copy link
Member

@LukaJCB LukaJCB commented Sep 1, 2017

No description provided.

@codecov-io
Copy link

codecov-io commented Sep 1, 2017

Codecov Report

Merging #1883 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1883      +/-   ##
==========================================
+ Coverage   94.97%   94.97%   +<.01%     
==========================================
  Files         241      241              
  Lines        4199     4202       +3     
  Branches      106      112       +6     
==========================================
+ Hits         3988     3991       +3     
  Misses        211      211
Impacted Files Coverage Δ
core/src/main/scala/cats/data/NonEmptyVector.scala 100% <100%> (ø) ⬆️
core/src/main/scala/cats/data/NonEmptyList.scala 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d85cc86...badbf53. Read the comment docs.

johnynek and others added 29 commits September 1, 2017 15:12
* Added more implementations of map2Eval

I hit a case where I wanted laziness with map2Eval and realized in many
cases where we could have it, we do not. There were all the cases I saw
that we missed, which I hope our existing tests cover.

* Add tests for map2 and map2Eval

* remove incorrect instances
I've verified this with `bench/jmh:run` which still runs benchmarks :)
This is a proposal to add `origami` to the list of Typelevel projects. However if I remember correctly one of the criteria for being part of the system is to have a compiler-checked documentation which doesn't exist anymore (it was there on a previous version of the library). I will add it and maybe then we can merge in this PR? Is anything else needed?
This is for your consideration. Grafter uses `Reader` instances for component dependencies and `Eval` for actions starting-up the application graph.
…el#1852)

sbt-ghpages is a dependency of sbt-microsites and therefore does
not need to be listed explicitly as plugin. We have the same situation
with tut whose keys we are using in our `build.sbt` but do not explicitly
list in `plugins.sbt`.
* Bump scalatest version to 3.0.3

* Bump scalatest version to 3.0.4
Both settings check if the `TRAVIS` environment variable is set.
`doctestWithDependencies` has been removed upstream since the plugin
does not automatically add dependencies anymore.
Alistair-Johnson and others added 6 commits September 1, 2017 15:12
* Limit runtime of monad loop tests

* Do not use Either syntax in monad loops

* Add iterateWhileM and iterateUntilM

* Implement iterateWhile in terms of iterateWhileM
Using parentheses for disabling tasks with sbt 1.0 will issue the
following deprecation warning:
```
warning: Adaptation of argument list by inserting () is deprecated: this
is unlikely to be what you want.
        signature: DefinableTask.:=(v: S): sbt.Def.Setting[sbt.Task[S]]
  given arguments: <none>
 after adaptation: DefinableTask.:=((): Unit)
  publish := (),
          ^
```

This can be fixed with curly braces which is also perfectly fine with
sbt 0.13. So this can be merged right now and will make the actual
upgrade to sbt 1.0 easier.
@LukaJCB LukaJCB force-pushed the add-zip-with branch 2 times, most recently from 4b500d0 to badbf53 Compare September 1, 2017 13:23
@LukaJCB LukaJCB closed this Sep 1, 2017
@LukaJCB LukaJCB deleted the add-zip-with branch September 1, 2017 13:27
@LukaJCB LukaJCB changed the title Add zipWith to NonEmptyList and NonEmptyVector Add zipWith Sep 1, 2017
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.