search

BraveyJS / Bravey

A simple JavaScript NLP-like library to help you creating your own bot.
https://braveyjs.github.io/
MIT License
78 stars 6 forks source link

Unnecessary console.warn() statements #14

Closed xrysanthos closed 6 years ago

xrysanthos commented 7 years ago

There are a number of console.warn() statements that screw up the logging flow of the integrating component. More specifically within the implementation of the Fuzzy.addDocument() method. Since console is a core JS module, the only way to bypass this effect is by short-wiring the warn() method.

nippur72 commented 6 years ago

I have the same issue here, I ship it to the browser but the console is flooded with warns.

Can I submit a PR for this? Do you want a switch or we can just remove them?

BraveyJS commented 6 years ago

Switched to info statements. Hopes it helps :)