scalacenter / scala-syntax

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

Improve coverage #20

Closed MasseGuillaume closed 6 years ago

MasseGuillaume commented 6 years ago

Boom 100%.

Bugs found with this PR:

I also removed some dead code.

codecov[bot] commented 6 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@04ecf2f). Click here to learn what that means. The diff coverage is 90.74%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #20   +/-   ##
=========================================
  Coverage          ?   68.67%           
=========================================
  Files             ?       16           
  Lines             ?      996           
  Branches          ?       74           
=========================================
  Hits              ?      684           
  Misses            ?      312           
  Partials          ?        0
Impacted Files Coverage Δ
...rc/main/scala/org/scalafmt/internal/TokenOps.scala 80% <ø> (ø)
.../main/scala/org/scalafmt/internal/TreeDocOps.scala 99.22% <100%> (ø)
...main/scala/org/scalafmt/internal/TreePrinter.scala 100% <100%> (ø)
.../main/scala/org/scalafmt/internal/ScalaToken.scala 100% <100%> (ø)
...cala/org/scalafmt/tests/BaseScalaPrinterTest.scala 74.28% <80.76%> (ø)

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 04ecf2f...15332d7. Read the comment docs.

MasseGuillaume commented 6 years ago

@olafurpg ^ so pretty!