vegas-viz / Vegas

The missing MatPlotLib for Scala + Spark
MIT License
730 stars 99 forks source link

Turn on warning unused imports and remove unused imports #149

Closed oshikiri closed 5 years ago

codecov-io commented 5 years ago

Codecov Report

Merging #149 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #149   +/-   ##
=====================================
  Coverage       0%     0%           
=====================================
  Files          27     27           
  Lines        1906   1906           
  Branches        5      5           
=====================================
  Misses       1906   1906
Impacted Files Coverage Δ
...c/main/scala/vegas/render/StaticHTMLRenderer.scala 0% <ø> (ø) :arrow_up:
...e/src/main/scala/vegas/render/WindowRenderer.scala 0% <ø> (ø) :arrow_up:
core/src/main/scala/vegas/DSL/ConfigDSL.scala 0% <ø> (ø) :arrow_up:
flink/src/main/scala/vegas/flink/Flink.scala 0% <ø> (ø) :arrow_up:
...src/main/scala/vegas/render/BaseHTMLRenderer.scala 0% <ø> (ø) :arrow_up:
core/src/main/scala/vegas/spec/package.scala 0% <ø> (ø) :arrow_up:
core/src/main/scala/vegas/DSL/package.scala 0% <ø> (ø) :arrow_up:
...src/main/scala/vegas/macros/ShowRenderMacros.scala 0% <ø> (ø) :arrow_up:
core/src/main/scala/vegas/package.scala 0% <ø> (ø) :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 1496432...b4b738a. Read the comment docs.

oshikiri commented 5 years ago

@aishfenton @dbtsai @rgbkrk Could you please review this?

rgbkrk commented 5 years ago

Looks like a great cleanup. I'll trust in you and CI for the validity of removals. 😄

oshikiri commented 5 years ago

@rgbkrk Thanks!!