Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

Update dependencies #109

Merged
merged 2 commits into from
Apr 27, 2016
Merged

Update dependencies #109

merged 2 commits into from
Apr 27, 2016

Conversation

jasonLaster
Copy link
Contributor

Updating the non-react dependencies. shouldn't be anything major in here.

@jlongster
Copy link
Contributor

Sounds good. We can just remove seamless-immutable as well, you can do that here if you want?

@jasonLaster jasonLaster merged commit 5062e5c into firefox-devtools:master Apr 27, 2016
@seflless
Copy link

@jlongster After reading a recent video of yours explaining the issues you were having using immutablejs and mixing with native components. I was going to suggest trying out seamless, but looks like you removed it here. What was your take on it?

Do you have a good write up somewhere about where your current thinking is on how to reduce confusion in mixing immutable and regular json structures? It's just the type system embracement that made you able to mix it better?

I am about to start a similar project and have been deeply pondering this issue of how far to go into using immutablejs, I'd prefer to use native structures but have guarantees that no code ended up being immutable unless explicitly doing so.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants