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/pride | |
| 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/pride')
| -rw-r--r-- | bot/exts/pride/drag_queen_name.py | 1 | ||||
| -rw-r--r-- | bot/exts/pride/pride_anthem.py | 1 | ||||
| -rw-r--r-- | bot/exts/pride/pride_avatar.py | 1 | ||||
| -rw-r--r-- | bot/exts/pride/pride_facts.py | 1 |
4 files changed, 0 insertions, 4 deletions
diff --git a/bot/exts/pride/drag_queen_name.py b/bot/exts/pride/drag_queen_name.py index 43813fbd..95297745 100644 --- a/bot/exts/pride/drag_queen_name.py +++ b/bot/exts/pride/drag_queen_name.py @@ -30,4 +30,3 @@ class DragNames(commands.Cog): def setup(bot: commands.Bot) -> None: """Cog loader for drag queen name generator.""" bot.add_cog(DragNames(bot)) - log.info("Drag queen name generator cog loaded!") diff --git a/bot/exts/pride/pride_anthem.py b/bot/exts/pride/pride_anthem.py index b0c6d34e..186c5fff 100644 --- a/bot/exts/pride/pride_anthem.py +++ b/bot/exts/pride/pride_anthem.py @@ -55,4 +55,3 @@ class PrideAnthem(commands.Cog): def setup(bot: commands.Bot) -> None: """Cog loader for pride anthem.""" bot.add_cog(PrideAnthem(bot)) - log.info("Pride anthems cog loaded!") diff --git a/bot/exts/pride/pride_avatar.py b/bot/exts/pride/pride_avatar.py index 85e49d5c..3f9878e3 100644 --- a/bot/exts/pride/pride_avatar.py +++ b/bot/exts/pride/pride_avatar.py @@ -142,4 +142,3 @@ class PrideAvatar(commands.Cog): def setup(bot: commands.Bot) -> None: """Cog load.""" bot.add_cog(PrideAvatar(bot)) - log.info("PrideAvatar cog loaded") diff --git a/bot/exts/pride/pride_facts.py b/bot/exts/pride/pride_facts.py index 2db8f5c2..bd9c087b 100644 --- a/bot/exts/pride/pride_facts.py +++ b/bot/exts/pride/pride_facts.py @@ -104,4 +104,3 @@ class PrideFacts(commands.Cog): def setup(bot: commands.Bot) -> None: """Cog loader for pride facts.""" bot.add_cog(PrideFacts(bot)) - log.info("Pride facts cog loaded!") |