We don’t have time for code reviews

Re:We don’t have time for code reviews by junto on Wednesday October 23, 2013 @10:17AM (#6598851)

As I said in another thread, always use console.log('message') and not alert('message').

Then for production you can always do this at the top of your main JavaScript include:

  var console = {};
  console.log = function(){};

[ Parent | on HN ]

Re:We don’t have time for code reviews by myke_cameron on Wednesday October 23, 2013 @10:24AM (#6598894)

I think a lot of people like using alert() for debugging because it posses execution. Of course, so do break-points / "debugger" statements.

Git pre-commit hooks for alert are helpful since few and far between are the circumstances where an alert is actually wanted in production. While you're at it you can add one for whatever set of profane terms, if you roll like that.

[ Parent | on HN ]