search

scalacenter / scala-syntax

Scalameta pretty printer for better Scalafix refactorings
Apache License 2.0
17 stars 6 forks source link

Fix Tokens.binarySearch & Ordering. #38

Closed MasseGuillaume closed 6 years ago

MasseGuillaume commented 6 years ago

The ordering was a bit tricky.

codecov[bot] commented 6 years ago

Codecov Report

Merging #38 into master will decrease coverage by 0.79%. The diff coverage is 68.42%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master      #38     +/-   ##
=========================================
- Coverage   87.87%   87.07%   -0.8%     
=========================================
  Files          24       25      +1     
  Lines        1039     1068     +29     
  Branches       37       54     +17     
=========================================
+ Hits          913      930     +17     
- Misses        126      138     +12
Impacted Files Coverage Δ
...rc/main/scala/org/scalafmt/internal/TokenOps.scala 54.54% <0%> (-31.17%) :arrow_down:
...c/main/scala/org/scalafmt/internal/Searching.scala 72.72% <72.72%> (ø)
...c/main/scala/org/scalafmt/internal/TokensOps.scala 86.84% <78.26%> (-13.16%) :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 314704e...145983f. Read the comment docs.