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.

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

Nuke Immutable Collections #453

Merged
merged 3 commits into from
Apr 16, 2014
Merged

Nuke Immutable Collections #453

merged 3 commits into from
Apr 16, 2014

Conversation

anaisbetts
Copy link
Contributor

Save a dependency and hopefully make our library more compatible with stuff /cc @hmemcpy

@hmemcpy
Copy link

hmemcpy commented Apr 15, 2014

@shiftkey
Copy link
Member

Given how much we actually depend on this, this seems like a good thing.

@haacked @trsneed thoughts?

@haacked
Copy link
Contributor

haacked commented Apr 15, 2014

Given how much we actually depend on this, this seems like a good thing.

Do you mean "given how little we depend on this?" If so, I agree. Make it so.

@shiftkey
Copy link
Member

@haacked yeah, that's what my brain meant to say

@trsneed
Copy link
Contributor

trsneed commented Apr 15, 2014

👍

@haacked
Copy link
Contributor

haacked commented Apr 15, 2014

Ok, I'll hit the merge button after we get the CI running again.

haacked added a commit that referenced this pull request Apr 16, 2014
@haacked haacked merged commit d479f2d into master Apr 16, 2014
@haacked haacked deleted the rip-immutable branch April 16, 2014 20:30
@haacked
Copy link
Contributor

haacked commented Apr 16, 2014

Thanks @paulcbetts!

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