search

tumblr / colossus

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

Remove some implicit #560

Closed benblack86 closed 7 years ago

benblack86 commented 7 years ago

I opened a similar PR before, but now will less changes :)

codecov-io commented 7 years ago

Codecov Report

Merging #560 into develop will increase coverage by 0.02%. The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #560      +/-   ##
===========================================
+ Coverage    84.83%   84.85%   +0.02%     
===========================================
  Files           98       98              
  Lines         4313     4319       +6     
  Branches       346      347       +1     
===========================================
+ Hits          3659     3665       +6     
  Misses         654      654
Impacted Files Coverage Δ
...c/main/scala/colossus/protocols/http/package.scala 33.33% <ø> (ø) :arrow_up:
...in/scala/colossus/protocols/memcache/package.scala 100% <ø> (ø) :arrow_up:
.../main/scala/colossus/protocols/redis/package.scala 16.66% <ø> (ø) :arrow_up:
...s/src/main/scala/colossus/service/ServiceDSL.scala 85.41% <100%> (+2.08%) :arrow_up:

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 6348231...5b3033c. Read the comment docs.