krisk / Fuse

Lightweight fuzzy-search, in JavaScript
https://fusejs.io/
Apache License 2.0
18.1k stars 766 forks source link

Fix for IE11 (ES5) compatibility #294

Closed favna closed 5 years ago

favna commented 5 years ago

This fix is to cover the PR feedback given by @StephenAWalsh on my previous PR (#291).

It was as simple as running the JS through babel's babel-loader rather than having it use whatever it does by default (I honestly never delved into it)

It should be ES5 compatible now, I ran it through es-check:

$ es-check es5 ./dist/fuse.js
ES-Check: there were no ES version matching errors!  �

(and manually confirmed the arrow functions are gone)


I also updated the Jest dependencies in ebfdc6d since they were showing as having updates available and might as well stay updated I'd say.

StephenAWalsh commented 5 years ago

Thank you!!