mdedetrich / scalajson

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

Fix JNumber.apply(Array[Byte]) #25

Closed htmldoug closed 7 years ago

htmldoug commented 7 years ago
scala> "hello world".toCharArray.toString
res0: String = [C@5922cff3

scala> "hello world".toCharArray.mkString
res1: String = hello world
codecov-io commented 7 years ago

Codecov Report

Merging #25 into master will increase coverage by 0.15%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #25      +/-   ##
==========================================
+ Coverage   44.64%   44.79%   +0.15%     
==========================================
  Files           5        5              
  Lines         663      663              
  Branches      140      140              
==========================================
+ Hits          296      297       +1     
+ Misses        367      366       -1
Impacted Files Coverage Δ
jvm/src/main/scala/scalajson/ast/JValue.scala 73% <100%> (+1%) :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 d863974...5e217af. Read the comment docs.