search

mdedetrich / scalajson

ScalaJSON - JSON for Scala, currently contains minimal AST
BSD 3-Clause "New" or "Revised" License
55 stars 10 forks source link

be Scala 2.13 friendly #11

Closed SethTisue closed 7 years ago

SethTisue commented 7 years ago

caught by the Scala 2.13 community build (yup, there is one already)

codecov-io commented 7 years ago

Codecov Report

Merging #11 into master will increase coverage by 4.94%. The diff coverage is n/a.

@@            Coverage Diff            @@
##           master     #11      +/-   ##
=========================================
+ Coverage   42.15%   47.1%   +4.94%     
=========================================
  Files           7       7              
  Lines         714     639      -75     
  Branches      142     139       -3     
=========================================
  Hits          301     301              
+ Misses        413     338      -75
Impacted Files Coverage Δ
jvm/src/main/scala/scala/json/ast/JValue.scala 0% <0%> (-69.91%) :x:
.../src/main/scala/scala/json/ast/unsafe/JValue.scala 0% <0%> (-60.37%) :x:
.../src/main/scala/scala/json/ast/unsafe/JValue.scala 60.36% <0%> (+60.36%) :white_check_mark:
js/src/main/scala/scala/json/ast/JValue.scala 69.9% <0%> (+69.9%) :white_check_mark:

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 ca86a78...31d4d24. Read the comment docs.

mdedetrich commented 7 years ago

Thanks for the PR!

SethTisue commented 7 years ago 
[info] Project scala-json-ast----------: SUCCESS (project rebuilt ok)

👍