search

JeffreySarnoff / NamedTupleTools.jl

some utilities for working with NamedTuples
MIT License
80 stars 11 forks source link

Error in merge_recursive doc string? #41

Closed danielmatz closed 2 years ago

danielmatz commented 2 years ago

The doc string for merge_recursive says that it propagates the value from the leftmost NamedTuple argument:

https://github.com/JeffreySarnoff/NamedTupleTools.jl/blob/9eedb5da218f60296429372e423c9c18c9d502f5/src/NamedTupleTools.jl#L317

However, the examples given and the code itself appear to propagate the value from the rightmost argument. And that would also match the convention in merge.

Is this just a typo?

JeffreySarnoff commented 2 years ago

yes. thank you.

JeffreySarnoff commented 2 years ago

actually hold on ...

JeffreySarnoff commented 2 years ago

ok revised, pending merge