Nadrieril / dhall-rust

Maintainable configuration files, for Rust users
Other
303 stars 27 forks source link

Rework import caching and resolution #192

Closed Nadrieril closed 3 years ago

Nadrieril commented 3 years ago

The import resolution and caching code was quite complicated: higher-level functions when not really needed, unidiomatic error handling, bad separation of concerns. This loses many unit tests, but imho it's worth it because the new code is so much clearer than the old code+unit tests. Might add some back as needed one day.