Skip to content

Conversation

@tpolecat
Copy link
Member

@tpolecat tpolecat commented Feb 27, 2021

Also disables eviction checking for now.

Subsumes #351

@codecov-io
Copy link

codecov-io commented Feb 27, 2021

Codecov Report

Merging #392 (1f94816) into ce3 (2d5a9e5) will increase coverage by 0.25%.
The diff coverage is 87.09%.

Impacted file tree graph

@@            Coverage Diff             @@
##              ce3     #392      +/-   ##
==========================================
+ Coverage   84.81%   85.07%   +0.25%     
==========================================
  Files         107      107              
  Lines        1291     1293       +2     
  Branches       26       22       -4     
==========================================
+ Hits         1095     1100       +5     
+ Misses        196      193       -3     
Impacted Files Coverage Δ
modules/core/src/main/scala/PreparedQuery.scala 21.56% <ø> (ø)
...les/core/src/main/scala/net/protocol/Startup.scala 93.54% <ø> (ø)
modules/core/src/main/scala/SSL.scala 41.17% <33.33%> (ø)
modules/core/src/main/scala/Session.scala 49.12% <100.00%> (-0.88%) ⬇️
...ules/core/src/main/scala/net/BitVectorSocket.scala 100.00% <100.00%> (+10.00%) ⬆️
...ore/src/main/scala/net/BufferedMessageSocket.scala 100.00% <100.00%> (ø)
...odules/core/src/main/scala/net/MessageSocket.scala 100.00% <100.00%> (ø)
modules/core/src/main/scala/net/Protocol.scala 81.25% <100.00%> (ø)
...dules/core/src/main/scala/net/SSLNegotiation.scala 80.00% <100.00%> (ø)
...ore/src/main/scala/net/AbstractMessageSocket.scala 83.33% <0.00%> (+33.33%) ⬆️

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 2d5a9e5...1f94816. Read the comment docs.

@tpolecat tpolecat merged commit f0e9b62 into ce3 Feb 27, 2021
@tpolecat tpolecat deleted the mpilquist-topic/fs2-network-refactor branch February 27, 2021 22:23
@tpolecat tpolecat restored the mpilquist-topic/fs2-network-refactor branch March 2, 2021 17:36
@mpilquist mpilquist mentioned this pull request Sep 22, 2022
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.

4 participants