tumblr / colossus

I/O and Microservice library for Scala
Apache License 2.0
1.14k stars 96 forks source link

Generic clients #535

Closed benblack86 closed 7 years ago

benblack86 commented 7 years ago

Added back information on generic clients. I'm trying to learn more about higher kinded types so this now makes sense! Whilst I was there, I changed a few flatMaps tomaps since nothing needed to be flatted.

codecov-io commented 7 years ago

Codecov Report

Merging #535 into master will increase coverage by 4.11%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #535      +/-   ##
==========================================
+ Coverage   80.62%   84.74%   +4.11%     
==========================================
  Files         117       98      -19     
  Lines        4512     4293     -219     
  Branches      359      357       -2     
==========================================
  Hits         3638     3638              
+ Misses        874      655     -219
Impacted Files Coverage Δ
...olossus-docs/src/main/scala/RedisRetryClient.scala
colossus-docs/src/main/scala/HttpService.scala
colossus-docs/src/main/scala/Fibonacci.scala
...lossus-docs/src/main/scala/MetricTickExample.scala
colossus-docs/src/main/scala/Metrics.scala
colossus-docs/src/main/scala/FutureClient.scala
colossus-docs/src/main/scala/HttpService2.scala
colossus-docs/src/main/scala/Fibonacci2.scala
colossus-docs/src/main/scala/RedisClient.scala
colossus-docs/src/main/scala/Fibonacci3.scala
... and 7 more

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 ad38333...b7ba29a. Read the comment docs.

benblack86 commented 7 years ago

Added codecov.yml

benblack86 commented 7 years ago

Also bumping SBT version since the previous version was causing problems with IntelliJ.