search

rt2zz / redux-action-buffer

Buffer Redux Actions
MIT License
79 stars 6 forks source link

Remove arrow functions #4

Closed Klathmon closed 8 years ago

Klathmon commented 8 years ago

You release redux-persist in ES5 (it looks like compiled with babel), however this module has arrow functions on one line.

This was blowing up my build when i added it as uglify-js still doesn't understand ES6 (or at least it doesn't in setup i'm using).

This is just a quick PR that removes the 3 inline arrow functions for "regular" functions which makes this fully ES5.

I ran standard lint so it should pass that, and i've already worked around the issue in my project, so don't feel like you need to rush to merge this.

rt2zz commented 8 years ago

great!

villavicencio commented 8 years ago

Hi @rt2zz, any chance you can bump the version so we can pull this change down? Looks like 1.0.0 still has arrow functions.

rt2zz commented 8 years ago

👍 done