aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar kwzrd <[email protected]>2020-06-19 17:55:20 +0200
committerGravatar kwzrd <[email protected]>2020-06-19 17:56:51 +0200
commit8a0263f5a591be51e74c2b26369f74c6d8dfee09 (patch)
treebc4d259d4658013d7a18c59c530feeb58d04c70e
parentIncidents: review log levels; use `trace` where appropriate (diff)
Incidents: remove broad noqa
This was originally in place to silence a PyCharm warning regarding accessing the private attributes. However, since there is no corresponding error code to specify, the noqa will silence any linter warning, which is potentially dangerous, and seems to be bad practice.
-rw-r--r--bot/cogs/moderation/incidents.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/bot/cogs/moderation/incidents.py b/bot/cogs/moderation/incidents.py
index da04c7d0d..c733607e6 100644
--- a/bot/cogs/moderation/incidents.py
+++ b/bot/cogs/moderation/incidents.py
@@ -265,7 +265,7 @@ class Incidents(Cog):
"""
await self.bot.wait_until_guild_available() # First make sure that the cache is ready
log.trace(f"Resolving message for: {message_id=}")
- message: discord.Message = self.bot._connection._get_message(message_id) # noqa: Private attribute
+ message: discord.Message = self.bot._connection._get_message(message_id)
if message is not None:
log.trace("Message was found in cache")