search

mozilla / scanjs

[DEPRECATED] Static analysis tool for javascript code.
Other
428 stars 39 forks source link

removing console.logs #123

Closed pauljt closed 10 years ago

pwnetrationguru commented 10 years ago

Should we just delete them? I don't see any advantage in just commenting them out.

That being said, idc too much, so LGTM if you want to merge as is.

mozfreddyb commented 10 years ago

We should either delete them of surround them with a check for a global variable called DEBUG, e.g. if (DEBUG) { console.log(...); }

pauljt commented 10 years ago

Just delete imho...

On 22 Apr 2014, at 7:43 pm, Frederik notifications@github.com wrote:

We should either delete them of surround them with a check for a global variable called DEBUG, e.g. if (DEBUG) { console.log(...); }

— Reply to this email directly or view it on GitHub.

pwnetrationguru commented 10 years ago

Deleted and merging.