arunoda / react-komposer

Feed data into React components by composing containers.
MIT License
733 stars 70 forks source link

Why the MobX dependency? #115

Closed SachaG closed 7 years ago

SachaG commented 8 years ago

Since React Komposer can be used with Meteor, Redux, MobX, etc. why does it depend on MobX specifically?

https://github.com/kadirahq/react-komposer/blob/master/package.json#L55

Isn't it a problem to make every app that uses React Komposer load MobX, even if they don't need it?

markshust commented 8 years ago

Correct, this should be a devDependency

arunoda commented 8 years ago

I'm going to handle this in a better way with 2.x https://github.com/kadirahq/react-komposer/issues/123

arunoda commented 7 years ago

Just released 2.x and now we don't ship mobx.