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.

Sign up for GitHub

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

Jump to bottom

Support merging top-level arrays #209

Closed

heaths opened this issue May 16, 2022 · 1 comment · Fixed by #210

Contributor

heaths commented May 16, 2022

It would be great if mergo supported top-level arrays. Seems it actually works if you expand the condition here:

https://github.com/imdario/mergo/blob/fd3dfc91a2151957f2fe870772747a469bc89ced/mergo.go#L68

Contributor Author

heaths commented May 16, 2022

Would be helpful for cli/cli#1268

heaths added a commit to heaths/mergo that referenced this issue


          Support slices of same type

200f90d

Fixes darccio#209

heaths mentioned this issue

Support slices of same type #210

Merged

vinbrucelu mentioned this issue

introduce a migration system for the config file ignite/cli#2470

Closed

darccio closed this as completed in #210

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment