-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
key merge strategy #145
Comments
So you are thinking if mergeStrategy is defined in the user config vs the c12 config? Then we could eventually put this back in defu to then use |
c12 config (controlled by |
I'll try to implement this and give it a shot for a POC 💯 Thanks for the feedback |
Implemented as simpler merger function so that end-users have full control #160 |
(this is a primary draft to explain the idea)
Related: #138, #87
Sometime default
defu
merging strategy is not ideal. We could allow a custom defu merger but it is not initiative.I was thinking to try with an object syntax like this:
(it could be backported to defu)
The text was updated successfully, but these errors were encountered: