EuIrcBot icon indicating copy to clipboard operation
EuIrcBot copied to clipboard

Published 20 hours ago • verified publisher icon wobscale

Update modules to use logger

Open LinuxMercedes opened this issue 8 years ago • 3 comments
trafficstars

#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

LinuxMercedes avatar Jun 27 '17 02:06 LinuxMercedes

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).

euank avatar Jun 28 '17 04:06 euank

Scrollback switched in #150

LinuxMercedes avatar Aug 02 '17 02:08 LinuxMercedes

dumbCommandModule switched in #190

iliana avatar Mar 06 '18 17:03 iliana