Roger-luo / Configurations.jl

Options & Configurations made easy.
https://configurations.rogerluo.dev/stable
MIT License
80 stars 12 forks source link

fix #55 #56

Closed Roger-luo closed 3 years ago

Roger-luo commented 3 years ago

we should use getfield over eval, also add downstream tests

codecov[bot] commented 3 years ago

Codecov Report

Merging #56 (9bbd2c8) into master (ef288c7) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #56   +/-   ##
=======================================
  Coverage   99.77%   99.77%           
=======================================
  Files           8        8           
  Lines         453      453           
=======================================
  Hits          452      452           
  Misses          1        1           
Impacted Files Coverage Δ
src/codegen.jl 100.00% <100.00%> (ø)

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 ef288c7...9bbd2c8. Read the comment docs.