LinuxMercedes commented 7 years ago

143 introduces a proper logger, rather than just using `console.log` everywhere. Modules need to be updated to use it.

This is a list of modules containing the string "console.log" that probably need to be converted to use a logger instead:

[ ] blerg
[ ] dota2
[x] dumbCommandModule
[ ] github
[ ] google
[ ] join
[ ] newegg
[x] nickserv
[ ] nicovideo
[ ] ping
[ ] qdb
[ ] que
[ ] quo
[ ] schedule
[x] scrollback
[ ] title
[ ] tumblr-insult
[ ] twitter
[ ] vimeo
[ ] weather
[ ] wolf
[ ] wotd
[ ] youtube

euank commented 7 years ago

I believe the examples in the testlog module are decently exhaustive.

The preferred method would be via ether the first (for anything with no changes in 'this', i.e. no arr.forEach or such), or the last (using 'init' to initialize a var log module variable).

LinuxMercedes commented 7 years ago

Scrollback switched in #150

iliana commented 6 years ago

dumbCommandModule switched in #190

wobscale / EuIrcBot

Update modules to use logger #144

143 introduces a proper logger, rather than just using `console.log` everywhere. Modules need to be updated to use it.

wobscale / EuIrcBot

Update modules to use logger #144

143 introduces a proper logger, rather than just using console.log everywhere. Modules need to be updated to use it.

143 introduces a proper logger, rather than just using `console.log` everywhere. Modules need to be updated to use it.