diff options
| author |  kwzrd <[email protected]> | 2020-03-28 16:33:20 +0100 |
|---|---|---|
| committer | kwzrd <[email protected]> | 2020-03-28 16:35:36 +0100 |
| commit | 5feabe642ed5b685c7aa9515bc70ff4ff8b278bc (patch) | |
| tree | 34643f1aad8fbd19f045bcaeffda897d58661599 /bot/exts/christmas/adventofcode.py | |
| parent | Merge master: sentry sdk, updated contributing docs (diff) | |
Deseasonify: log in `add_cog` rather than in each `setup`
The previous system required each extension's `setup` func to log
that the cog was loaded. This leads to inconsistent messages all
trying to convey the same thing, variable logger names in the output
file are difficult to read, and several extensions were not logging
at all.
By logging directly in the `add_cog` method, we reduce code repetition,
ensure consistent format, and remove the responsibility to remember
that a log should be made.
Diffstat (limited to 'bot/exts/christmas/adventofcode.py')
| -rw-r--r-- | bot/exts/christmas/adventofcode.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/bot/exts/christmas/adventofcode.py b/bot/exts/christmas/adventofcode.py index f7590e04..8f7aa13b 100644 --- a/bot/exts/christmas/adventofcode.py +++ b/bot/exts/christmas/adventofcode.py @@ -741,4 +741,3 @@ def _error_embed_helper(title: str, description: str) -> discord.Embed: def setup(bot: commands.Bot) -> None: """Advent of Code Cog load.""" bot.add_cog(AdventOfCode(bot)) - log.info("AdventOfCode cog loaded") |