Update Provider.js #358

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

Merged

gaearon merged 1 commit into reduxjs:master from koistya:patch-1

Apr 18, 2016

Contributor

koistya commented Apr 18, 2016

No description provided.


          Update Provider.js

c20ae48

Contributor

gaearon commented Apr 18, 2016

Any particular reason you prefer this style?

Contributor Author

koistya commented Apr 18, 2016 •

edited

Loading

Less variable declarations, smaller code, maybe a bit more efficient (?)

gaearon merged commit 1ef78c0 into reduxjs:master

Contributor

gaearon commented Apr 18, 2016

I wouldn’t think either is more efficient but I’m OK with this, it does read easier on eyes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet