tumblr / colossus

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

Merge master into develop #520

Closed benblack86 closed 7 years ago

benblack86 commented 7 years ago

@DanSimon

codecov-io commented 7 years ago

Codecov Report

Merging #520 into develop will decrease coverage by 0.03%. The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #520      +/-   ##
===========================================
- Coverage    84.73%   84.69%   -0.04%     
===========================================
  Files           98       98              
  Lines         4291     4294       +3     
  Branches       336      338       +2     
===========================================
+ Hits          3636     3637       +1     
- Misses         655      657       +2
Impacted Files Coverage Δ
...scala/colossus/protocols/websocket/Websocket.scala 91.15% <100%> (+0.07%) :arrow_up:
...sus/src/main/scala/colossus/service/Callback.scala 89.38% <100%> (+0.19%) :arrow_up:
...la/colossus/protocols/http/HttpRequestParser.scala 94.73% <100%> (ø) :arrow_up:
...rc/main/scala/colossus/service/ServiceServer.scala 91.75% <0%> (-1.04%) :arrow_down:
.../scala/colossus/metrics/collectors/Histogram.scala 91.91% <0%> (-0.74%) :arrow_down:

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 1304ac7...71bc62c. Read the comment docs.

DanSimon commented 7 years ago

👍