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.

8 files changed, 0 insertions, 8 deletions

diff --git a/bot/exts/valentines/be_my_valentine.py b/bot/exts/valentines/be_my_valentine.py
index 1e883d21..4b0818da 100644
--- a/bot/exts/valentines/be_my_valentine.py
+++ b/bot/exts/valentines/be_my_valentine.py
@@ -234,4 +234,3 @@ class BeMyValentine(commands.Cog):
 def setup(bot: commands.Bot) -> None:
     """Be my Valentine Cog load."""
     bot.add_cog(BeMyValentine(bot))
-    log.info("BeMyValentine cog loaded")
diff --git a/bot/exts/valentines/lovecalculator.py b/bot/exts/valentines/lovecalculator.py
index 03d3d7d5..e11e062b 100644
--- a/bot/exts/valentines/lovecalculator.py
+++ b/bot/exts/valentines/lovecalculator.py
@@ -101,4 +101,3 @@ class LoveCalculator(Cog):
 def setup(bot: commands.Bot) -> None:
     """Love calculator Cog load."""
     bot.add_cog(LoveCalculator(bot))
-    log.info("LoveCalculator cog loaded")
diff --git a/bot/exts/valentines/movie_generator.py b/bot/exts/valentines/movie_generator.py
index ce1d7d5b..0843175a 100644
--- a/bot/exts/valentines/movie_generator.py
+++ b/bot/exts/valentines/movie_generator.py
@@ -60,4 +60,3 @@ class RomanceMovieFinder(commands.Cog):
 def setup(bot: commands.Bot) -> None:
     """Romance movie Cog load."""
     bot.add_cog(RomanceMovieFinder(bot))
-    log.info("RomanceMovieFinder cog loaded")
diff --git a/bot/exts/valentines/myvalenstate.py b/bot/exts/valentines/myvalenstate.py
index 0256c39a..7d8737c4 100644
--- a/bot/exts/valentines/myvalenstate.py
+++ b/bot/exts/valentines/myvalenstate.py
@@ -84,4 +84,3 @@ class MyValenstate(commands.Cog):
 def setup(bot: commands.Bot) -> None:
     """Valenstate Cog load."""
     bot.add_cog(MyValenstate(bot))
-    log.info("MyValenstate cog loaded")
diff --git a/bot/exts/valentines/pickuplines.py b/bot/exts/valentines/pickuplines.py
index 8b2c9822..74c7e68b 100644
--- a/bot/exts/valentines/pickuplines.py
+++ b/bot/exts/valentines/pickuplines.py
@@ -42,4 +42,3 @@ class PickupLine(commands.Cog):
 def setup(bot: commands.Bot) -> None:
     """Pickup lines Cog load."""
     bot.add_cog(PickupLine(bot))
-    log.info('PickupLine cog loaded')
diff --git a/bot/exts/valentines/savethedate.py b/bot/exts/valentines/savethedate.py
index e0bc3904..ac38d279 100644
--- a/bot/exts/valentines/savethedate.py
+++ b/bot/exts/valentines/savethedate.py
@@ -39,4 +39,3 @@ class SaveTheDate(commands.Cog):
 def setup(bot: commands.Bot) -> None:
     """Save the date Cog Load."""
     bot.add_cog(SaveTheDate(bot))
-    log.info("SaveTheDate cog loaded")
diff --git a/bot/exts/valentines/valentine_zodiac.py b/bot/exts/valentines/valentine_zodiac.py
index c8d77e75..1a1273aa 100644
--- a/bot/exts/valentines/valentine_zodiac.py
+++ b/bot/exts/valentines/valentine_zodiac.py
@@ -55,4 +55,3 @@ class ValentineZodiac(commands.Cog):
 def setup(bot: commands.Bot) -> None:
     """Valentine zodiac Cog load."""
     bot.add_cog(ValentineZodiac(bot))
-    log.info("ValentineZodiac cog loaded")
diff --git a/bot/exts/valentines/whoisvalentine.py b/bot/exts/valentines/whoisvalentine.py
index b8586dca..4ca0289c 100644
--- a/bot/exts/valentines/whoisvalentine.py
+++ b/bot/exts/valentines/whoisvalentine.py
@@ -50,4 +50,3 @@ class ValentineFacts(commands.Cog):
 def setup(bot: commands.Bot) -> None:
     """Who is Valentine Cog load."""
     bot.add_cog(ValentineFacts(bot))
-    log.info("ValentineFacts cog loaded")