manaflair / redux-batch

Enhance your Redux store to support batched actions
174 stars 12 forks source link

Patch for issue #11 #12

Closed williamdes closed 6 years ago

williamdes commented 6 years ago

See issue #11

williamdes commented 6 years ago

@arcanis What do you think about this PR ? Need more information ?

arcanis commented 6 years ago

@williamdes can you share the erroneous compiled version of the file and open an issue on the Uglifyjs tracker? If it's a bug in uglify, they sure should be made aware of it. Once done I'll merge your PR 🙂

williamdes commented 6 years ago

@arcanis I think it will be quite impossible to debug, I created a test project and I can reproduce the bug only with the folder node_modules containing the bug. So in fact what I sent in this PR is what webpack w/ Uglifyjs would do when bundling. I reinstalled my node_modules and everything is back to normal. Sorry for this false-positive issue. NPM had done some think strange to the node_modules at the installation time...