enhancement: allow ac_merge to be a callable for user-defined merge strategies

ssato / python-anyconfig

Python library provides common APIs to load and dump configuration files in various formats

MIT License

277 stars 31 forks source link

enhancement: allow ac_merge to be a callable for user-defined merge strategies #77

Closed csboling closed 7 years ago

csboling commented 7 years ago

Allows a callable to be passed in as the ac_merge option to e.g. load. Seemed like the simplest way to allow client code to customize the merge strategy.

coveralls commented 7 years ago

Coverage decreased (-0.06%) to 93.045% when pulling 174dadc65dd728a39453d7181814087d7bd6f707 on csboling:custom-merge-strat into c372b9803a8c9bdb84ef7764db734ee437c9d5cd on ssato:master.

coveralls commented 7 years ago

Coverage decreased (-0.06%) to 93.045% when pulling 174dadc65dd728a39453d7181814087d7bd6f707 on csboling:custom-merge-strat into c372b9803a8c9bdb84ef7764db734ee437c9d5cd on ssato:master.

coveralls commented 7 years ago

Coverage increased (+0.009%) to 93.11% when pulling 78defe925675f93f56f5fd129e95d74bfff313b9 on csboling:custom-merge-strat into c372b9803a8c9bdb84ef7764db734ee437c9d5cd on ssato:master.

ssato commented 7 years ago

Thanks a lot for your neat PR!

Actually, this is one of what I've been wanted to improve and wondered how should be implemented for a while. Your patch looks great and good for me. I'll merge the PR.