Remove console.log()'s

Question

Remove console.log()'s

hollandThomas opened this issue 7 years ago · 2 comments

Hey, first of all thanks for making and open sourcing this library! :)

I was just skimming through the source code when I noticed a console.log here. Unless I'm missing something and this gets stripped in production, using console.log in production is considered harmful™ which is ironical, seeing that this package seeks to improve performance ;)

Thanks for looking into this!

Answer 1 · 2017-11-29T22:01:03.000Z

I'll remove them. Thanks for reporting!

Answer 2 · 2017-11-30T05:31:39.000Z

This is now done in v1.6.3. You can set debug = true like so matchQueryMiddleware(queryMapJson, true) to view console.logs.