search

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

Don't treat null strings specially #288

Closed Herringway closed 2 years ago

Herringway commented 2 years ago

Although null and "" have basically the same meaning in D, this isn't universal, and the general consensus amongst yaml implementations is that empty strings should be empty strings. Fixes #286.

codecov[bot] commented 2 years ago

Codecov Report

Merging #288 (fa2fbb6) into master (e525300) will decrease coverage by 0.00%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #288      +/-   ##
==========================================
- Coverage   96.03%   96.03%   -0.01%     
==========================================
  Files          28       28              
  Lines        4087     4085       -2     
==========================================
- Hits         3925     3923       -2     
  Misses        162      162
Impacted Files Coverage Δ
source/dyaml/representer.d 99.48% <100.00%> (-0.01%) :arrow_down:

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 e525300...fa2fbb6. Read the comment docs.