Closed markdegrootnl closed 5 years ago
Thank you! I will review your changes this Sunday.
Is there any reason why you skipped
config3
files intests/mocks
?I think that
config4
files should be renamed toconfig3
and tests should also be updated to use correct files.Anyway, thank you for this PR! I will merge it when I review it completely and you fix requested changes.
Config 3 is used in another test to check the case if a file does not exist.
Another implementation to add the ability to write a Config back to file as discussed in #107.
Most of the heavy lifting was already done by @filips123 in https://github.com/filips123/config/tree/writing however my implementation does not contain any breaking changes and comforms with the current state of the project.