Skip to content

Added map-merge-deep function#10

Open
Undistraction wants to merge 6 commits intoat-import:0.x.xfrom
Undistraction:map-merge-deep
Open

Added map-merge-deep function#10
Undistraction wants to merge 6 commits intoat-import:0.x.xfrom
Undistraction:map-merge-deep

Conversation

@Undistraction
Copy link

map-merge-deep combines two maps into a single map. The merge is recursive, meaning maps can contain other maps to an arbitrary depth. By default it will override existing key/values but this can be configured with a parameter.

map-merge-deep combines two maps into a single map. The merge is recursive, meaning maps can contain other maps to an arbitrary depth.
@Undistraction
Copy link
Author

Not sure why that's failing. Tests are passing at my end.

@ericfreese
Copy link

+1 hope this gets added.

@Snugug
Copy link
Member

Snugug commented Nov 12, 2014

Related to #9

@Snugug
Copy link
Member

Snugug commented Nov 12, 2014

It appears to be failing because the control file is missing a newline at the end.

@Snugug
Copy link
Member

Snugug commented Nov 12, 2014

Please also update the README with documentation on the new functionality

@Undistraction
Copy link
Author

@Snugug I'll get this updated tonight.

@calvinf
Copy link

calvinf commented May 19, 2015

+1, what's the status on this pull request?

@monsdroid
Copy link

+1

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants