search

scalacenter / scala-syntax

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

Fix ApplyUnary -(1.s) and -(-a) #22

Closed MasseGuillaume closed 6 years ago

olafurpg commented 6 years ago

I think it's fine to make the ApplyUnary case more complicated if that avoids leaking complexity into syntactic groups.

codecov[bot] commented 6 years ago

Codecov Report

Merging #22 into master will increase coverage by 0.15%. The diff coverage is 80%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
+ Coverage   86.21%   86.37%   +0.15%     
==========================================
  Files          15       15              
  Lines         791      800       +9     
  Branches       37       36       -1     
==========================================
+ Hits          682      691       +9     
  Misses        109      109
Impacted Files Coverage Δ
...main/scala/org/scalafmt/internal/TreePrinter.scala 100% <100%> (ø) :arrow_up:
.../main/scala/org/scalafmt/internal/TreeDocOps.scala 99.23% <100%> (ø) :arrow_up:
...scala/scala/meta/internal/fmt/SyntacticGroup.scala 64.86% <100%> (+2%) :arrow_up:
...rc/main/scala/org/scalafmt/internal/TokenOps.scala 83.33% <100%> (+2.25%) :arrow_up:
...ala/org/scalafmt/internal/TreeSyntacticGroup.scala 55.38% <25%> (ø) :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 4b40364...01bfa6c. Read the comment docs.