search

JuliaPhysics / PeriodicTable.jl

Periodic Table for Julians! :fire:
Other
111 stars 26 forks source link

rm json dependency #9

Closed stevengj closed 6 years ago

stevengj commented 6 years ago

This PR removes the JSON.jl dependency completely. Since the JSON data only needs to converted to Julia once, you might as well store the data in Julia format rather than JSON format.

codecov-io commented 6 years ago

Codecov Report

Merging #9 into master will increase coverage by 11.47%. The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master       #9       +/-   ##
===========================================
+ Coverage   80.35%   91.83%   +11.47%     
===========================================
  Files           1        1               
  Lines          56       49        -7     
===========================================
  Hits           45       45               
+ Misses         11        4        -7
Impacted Files Coverage Δ
src/PeriodicTable.jl 91.83% <0%> (+11.47%) :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 07d5030...62a8621. Read the comment docs.