dlang-community / D-YAML

YAML parser and emitter for the D programming language
https://dlang-community.github.io/D-YAML/
Boost Software License 1.0
119 stars 37 forks source link

Merge Node constructors #146

Closed Herringway closed 6 years ago

Herringway commented 6 years ago

Previously, the constructors for Node were separate because they couldn't all share the same attributes - nothrow, @safe, etc. Since attributes are automatically inferred for templates, this is no longer necessary.

Other notes:

codecov[bot] commented 6 years ago

Codecov Report

Merging #146 into master will increase coverage by <.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #146      +/-   ##
==========================================
+ Coverage   93.37%   93.38%   +<.01%     
==========================================
  Files          31       31              
  Lines        4331     4337       +6     
==========================================
+ Hits         4044     4050       +6     
  Misses        287      287
Impacted Files Coverage Δ
source/dyaml/node.d 91.66% <100%> (+0.06%) :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 28a6118...11d3f7c. Read the comment docs.