Skip to content

Added map-merge-deep function - #10

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

Undistraction wants to merge 6 commits into
at-import:0.x.xfrom
Undistraction:map-merge-deep

Conversation

@Undistraction

Copy link
Copy Markdown

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
Copy Markdown
Author

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

@ericfreese

Copy link
Copy Markdown

+1 hope this gets added.

@Snugug

Snugug commented Nov 12, 2014

Copy link
Copy Markdown
Member

Related to #9

@Snugug

Snugug commented Nov 12, 2014

Copy link
Copy Markdown
Member

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

@Snugug

Snugug commented Nov 12, 2014

Copy link
Copy Markdown
Member

Please also update the README with documentation on the new functionality

@Undistraction

Copy link
Copy Markdown
Author

@Snugug I'll get this updated tonight.

@calvinf

calvinf commented May 19, 2015

Copy link
Copy Markdown

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

@monsdroid

Copy link
Copy Markdown

+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