Remove console output by default

addyosmani / timing.js

Navigation Timing API measurement helpers

1.5k stars 114 forks source link

Remove console output by default #19

Closed amk221 closed 9 years ago

amk221 commented 9 years ago

This library is useful for collecting and reporting statistics in production, therefore it would be great if it didn't output to the console by default.

Fryie commented 9 years ago

+1! Also, the printSimpleTable seems to break in some browsers (e.g. IE).

In fact, I think it's harmful for a library to do side effects when being imported.

addyosmani commented 9 years ago

I'm personally fine with this change. LGTM.