Ngx-scanner: Remove log output

Created on 26 Feb 2018  路  4Comments  路  Source: zxing-js/ngx-scanner

I'd suggest that we remove the console.log and console.warn output or at least replace it with console.debug (browser-code-reader.ts)

enhancement

Most helpful comment

Are those needed?

I think all of them could be removed from builds, as they are only used in development of the library. If the library works, no one will use them. If someone would like to debug the library they will pull the repo and see the console.

All 4 comments

Are those needed?

I think all of them could be removed from builds, as they are only used in development of the library. If the library works, no one will use them. If someone would like to debug the library they will pull the repo and see the console.

Please, note that I'm publishing under a v0.0.0-**dev**.<hash> tag, I intend to remove these logs at the final version as well. 馃槃

Done.

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

Was this page helpful?
0 / 5 - 0 ratings