Skip to content
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

Recursive key change for Maps #48

Merged
merged 3 commits into from
Jul 31, 2015
Merged

Recursive key change for Maps #48

merged 3 commits into from
Jul 31, 2015

Conversation

mykewould
Copy link
Owner

  • Create basic functionality
  • Check with team on name and function placement (should it live some place else?)
  • Refactor
  • Enum.dmap

@CyrusNuevoDia CyrusNuevoDia changed the title Recurvise key change for Maps Recursive key change for Maps Jul 29, 2015
@CyrusNuevoDia
Copy link
Collaborator

Changing this to a recursive map for any Enum. And I'm trying my hand at it.

@mykewould
Copy link
Owner Author

A refactor provided by @druzn3k

https://gist.github.com/druzn3k/3719f32e992c63f9e7e5

As it's moving, that may or may not be useful.

@mykewould
Copy link
Owner Author

Waiting on dmap to land to close out issue.

mykewould added a commit that referenced this pull request Jul 31, 2015
@mykewould
Merge pull request #48 from mykewould/deep_key_change
edf2637 
Recursive key change for Maps
@mykewould mykewould merged commit edf2637 into master Jul 31, 2015
@mykewould mykewould deleted the deep_key_change branch July 31, 2015 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mykewould @CyrusNuevoDia