mdedetrich / scalajson

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

Make case classes/case objects final, fixes #27 #29

Closed mdedetrich closed 7 years ago

codecov-io commented 7 years ago

Codecov Report

Merging #29 into master will increase coverage by 0.15%. The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
+ Coverage   44.79%   44.94%   +0.15%     
==========================================
  Files           5        5              
  Lines         663      663              
  Branches      140      140              
==========================================
+ Hits          297      298       +1     
+ Misses        366      365       -1
Impacted Files Coverage Δ
js/src/main/scala/scalajson/ast/JValue.scala 0% <ø> (ø) :arrow_up:
jvm/src/main/scala/scalajson/ast/JValue.scala 74% <ø> (+1%) :arrow_up:
...s/src/main/scala/scalajson/ast/unsafe/JValue.scala 0% <ø> (ø) :arrow_up:
...m/src/main/scala/scalajson/ast/unsafe/JValue.scala 60.64% <66.66%> (ø) :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 2b096c2...23174e3. Read the comment docs.