search

mehcode / config-rs

⚙️ Layered configuration system for Rust applications (with strong support for 12-factor applications).
Apache License 2.0
2.43k stars 206 forks source link

Unify deser impl (redux) #488

Closed ijackson closed 8 months ago

ijackson commented 8 months ago

This is #338 reworked to apply to current main, and with the test case simplified as requested.

polarathene commented 8 months ago

I haven't checked yet, but heads-up prior to all the rework done recently to the ser.rs and de.rs, I had identified the cause of https://github.com/mehcode/config-rs/issues/461#issuecomment-1771963758 and fix. Although that's less obvious now.

Might be worth verifying if you've implicitly resolved that issue too, or added more drift 😅