From e4b8b47ad4d32142bbea09253382f32a993619a9 Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Mon, 29 Jan 2024 21:04:29 +0000 Subject: [PATCH 1/2] [meta] update pre-commit MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit updates: - [github.com/psf/black: 23.12.1 → 24.1.1](https://github.com/psf/black/compare/23.12.1...24.1.1) --- .pre-commit-config.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 5f3b56b0..47119eec 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -4,7 +4,7 @@ default_language_version: python: python3.11 repos: - repo: https://github.com/psf/black - rev: 23.12.1 + rev: 24.1.1 hooks: - id: black - repo: https://github.com/pycqa/isort From 83e32d2b92bf5bbbdafc4cc072c2dbffccdb93e1 Mon Sep 17 00:00:00 2001 From: "pre-commit-ci[bot]" <66853113+pre-commit-ci[bot]@users.noreply.github.com> Date: Mon, 29 Jan 2024 21:04:39 +0000 Subject: [PATCH 2/2] [meta] pre-commit styling fix --- adminutils/adminutils.py | 24 ++++--- captcha/commands/settings.py | 18 ++--- datautils/datautils.py | 96 +++++++++++++++++--------- datautils/menus.py | 6 +- godvilledata/godvilledata.py | 16 +++-- leveler/commands/lvladmin/badge.py | 6 +- leveler/commands/lvladmin/debugging.py | 8 ++- leveler/commands/lvlset/badge.py | 6 +- leveler/commands/top.py | 25 ++++--- leveler/exp.py | 16 +++-- leveler/menus/top.py | 8 ++- minecraftdata/minecraftdata.py | 26 +++---- steamcommunity/steamcommunity.py | 8 ++- 13 files changed, 159 insertions(+), 104 deletions(-) diff --git a/adminutils/adminutils.py b/adminutils/adminutils.py index 66849830..b905eb95 100644 --- a/adminutils/adminutils.py +++ b/adminutils/adminutils.py @@ -202,9 +202,11 @@ async def emoji_add(self, ctx, name: str, url: str, *roles: discord.Role): roles=roles, reason=get_audit_reason( ctx.author, - _("Restricted to roles: {}").format(", ".join(role.name for role in roles)) - if roles - else None, + ( + _("Restricted to roles: {}").format(", ".join(role.name for role in roles)) + if roles + else None + ), ), ) except discord.InvalidArgument: @@ -245,9 +247,11 @@ async def emote_steal( roles=roles, reason=get_audit_reason( ctx.author, - _("Restricted to roles: {}").format(", ".join(role.name for role in roles)) - if roles - else None, + ( + _("Restricted to roles: {}").format(", ".join(role.name for role in roles)) + if roles + else None + ), ), ) await ctx.tick() @@ -283,9 +287,11 @@ async def emoji_rename( roles=roles, reason=get_audit_reason( ctx.author, - _("Restricted to roles: ").format(", ".join(role.name for role in roles)) - if roles - else None, + ( + _("Restricted to roles: ").format(", ".join(role.name for role in roles)) + if roles + else None + ), ), ) except discord.Forbidden: diff --git a/captcha/commands/settings.py b/captcha/commands/settings.py index 6ae2235f..ba4326f3 100644 --- a/captcha/commands/settings.py +++ b/captcha/commands/settings.py @@ -308,14 +308,14 @@ async def add_roles(self, ctx: commands.Context, *roles: discord.Role): for role in roles: async with self.data.guild(ctx.guild).autoroles() as roles_list: if role.position >= ctx.me.top_role.position: - error[ - role.name - ] = "This role is higher than my highest role in the role hierarchy." + error[role.name] = ( + "This role is higher than my highest role in the role hierarchy." + ) continue if role.position >= ctx.author.top_role.position: - error[ - role.name - ] = "This role is higher than your own in the discord hierarchy." + error[role.name] = ( + "This role is higher than your own in the discord hierarchy." + ) continue if role.id not in roles_list: roles_list.append(role.id) @@ -356,9 +356,9 @@ async def remove_roles(self, ctx: commands.Context, *roles: discord.Role): async with self.data.guild(ctx.guild).autoroles() as roles_list: for role in roles: if role.position >= ctx.author.top_role.position: - error[ - role.name - ] = "This role is higher than your own in the discord hierarchy." + error[role.name] = ( + "This role is higher than your own in the discord hierarchy." + ) continue if role.id in roles_list: roles_list.remove(role.id) diff --git a/datautils/datautils.py b/datautils/datautils.py index 1b2d91df..470fbe5c 100644 --- a/datautils/datautils.py +++ b/datautils/datautils.py @@ -127,8 +127,12 @@ async def fetchwidget(self, ctx, *, server_id: int): em.add_field( name=_("Features"), value="\n".join(_(GUILD_FEATURES.get(f, f)) for f in guild.features).format( - banner=guild.banner and f" [🔗]({guild.banner_url_as(format='png')})" or "", - splash=guild.splash and f" [🔗]({guild.splash_url_as(format='png')})" or "", + banner=guild.banner + and f" [🔗]({guild.banner_url_as(format='png')})" + or "", + splash=guild.splash + and f" [🔗]({guild.splash_url_as(format='png')})" + or "", discovery=getattr(guild, "discovery_splash", None) and f" [🔗]({guild.discovery_splash_url_as(format='png')})" or "", @@ -263,35 +267,55 @@ async def sinfo(self, ctx, *, server: commands.GuildConverter = None): ) em.add_field( name=_("Verification level"), - value=_("None") - if server.verification_level == discord.VerificationLevel.none - else _("Low") - if server.verification_level == discord.VerificationLevel.low - else _("Medium") - if server.verification_level == discord.VerificationLevel.medium - else _("High") - if server.verification_level == discord.VerificationLevel.high - else _("Highest") - if server.verification_level == discord.VerificationLevel.extreme - else _("Unknown"), + value=( + _("None") + if server.verification_level == discord.VerificationLevel.none + else ( + _("Low") + if server.verification_level == discord.VerificationLevel.low + else ( + _("Medium") + if server.verification_level == discord.VerificationLevel.medium + else ( + _("High") + if server.verification_level == discord.VerificationLevel.high + else ( + _("Highest") + if server.verification_level == discord.VerificationLevel.extreme + else _("Unknown") + ) + ) + ) + ) + ), ) em.add_field( name=_("Explicit content filter"), - value=_("Don't scan any messages.") - if server.explicit_content_filter == discord.ContentFilter.disabled - else _("Scan messages from members without a role.") - if server.explicit_content_filter == discord.ContentFilter.no_role - else _("Scan messages sent by all members.") - if server.explicit_content_filter == discord.ContentFilter.all_members - else _("Unknown"), + value=( + _("Don't scan any messages.") + if server.explicit_content_filter == discord.ContentFilter.disabled + else ( + _("Scan messages from members without a role.") + if server.explicit_content_filter == discord.ContentFilter.no_role + else ( + _("Scan messages sent by all members.") + if server.explicit_content_filter == discord.ContentFilter.all_members + else _("Unknown") + ) + ) + ), ) em.add_field( name=_("Default notifications"), - value=_("All messages") - if server.default_notifications == discord.NotificationLevel.all_messages - else _("Only @mentions") - if server.default_notifications == discord.NotificationLevel.only_mentions - else _("Unknown"), + value=( + _("All messages") + if server.default_notifications == discord.NotificationLevel.all_messages + else ( + _("Only @mentions") + if server.default_notifications == discord.NotificationLevel.only_mentions + else _("Unknown") + ) + ), ) em.add_field(name=_("2FA admins"), value=bool_emojify(server.mfa_level)) if server.rules_channel: @@ -432,15 +456,19 @@ async def cinfo( changed_roles = sorted(channel.changed_roles, key=lambda r: r.position, reverse=True) em = discord.Embed( title=chat.escape(str(channel.name), formatting=True), - description=topic - if (topic := getattr(channel, "topic", None)) - else "\N{SPEECH BALLOON}: {} | \N{SPEAKER}: {} | \N{SATELLITE ANTENNA}: {}".format( - len(channel.text_channels), - len(channel.voice_channels), - len(channel.stage_channels), - ) - if isinstance(channel, discord.CategoryChannel) - else discord.Embed.Empty, + description=( + topic + if (topic := getattr(channel, "topic", None)) + else ( + "\N{SPEECH BALLOON}: {} | \N{SPEAKER}: {} | \N{SATELLITE ANTENNA}: {}".format( + len(channel.text_channels), + len(channel.voice_channels), + len(channel.stage_channels), + ) + if isinstance(channel, discord.CategoryChannel) + else discord.Embed.Empty + ) + ), color=await ctx.embed_color(), ) em.add_field(name=_("ID"), value=channel.id) diff --git a/datautils/menus.py b/datautils/menus.py index d78c9fba..d2157a4f 100644 --- a/datautils/menus.py +++ b/datautils/menus.py @@ -168,9 +168,9 @@ def __init__(self, entries): async def format_page(self, menu: ChannelsMenu, entries): e = discord.Embed( title="{}:".format(_(KNOWN_CHANNEL_TYPES[menu.channel_type][1])), - description=chat.box("\n".join(c.name for c in entries)) - if entries - else _("No channels"), + description=( + chat.box("\n".join(c.name for c in entries)) if entries else _("No channels") + ), ) e.set_footer( text=_("Page {}/{} • {}: {} • Total channels: {}").format( diff --git a/godvilledata/godvilledata.py b/godvilledata/godvilledata.py index 69274a4a..c3fb3c9d 100644 --- a/godvilledata/godvilledata.py +++ b/godvilledata/godvilledata.py @@ -95,9 +95,11 @@ async def godville(self, ctx, *, god: GodConverter): text_header = "{} и его {}\n{}\n".format( chat.bold(profile.god), chat.bold(profile.name), - chat.italics(chat.escape(profile.motto.strip(), formatting=True)) - if profile.motto - else chat.inline("Здесь ничего нет"), + ( + chat.italics(chat.escape(profile.motto.strip(), formatting=True)) + if profile.motto + else chat.inline("Здесь ничего нет") + ), ) if profile.arena_is_in_fight: text_header += "В сражении: {}\n".format(profile.fight_type_rus) @@ -212,9 +214,11 @@ async def godvillegame(self, ctx, *, godname: str): text_header = "{} and his {}\n{}\n".format( chat.bold(profile.god), chat.bold(profile.name), - chat.italics(chat.escape(profile.motto.strip(), formatting=True)) - if profile.motto - else chat.inline("Nothing here"), + ( + chat.italics(chat.escape(profile.motto.strip(), formatting=True)) + if profile.motto + else chat.inline("Nothing here") + ), ) if profile.arena_is_in_fight: text_header += "In fight: {}\n".format(profile.fight_type_rus) diff --git a/leveler/commands/lvladmin/badge.py b/leveler/commands/lvladmin/badge.py index fa82aa84..dcbc8f08 100644 --- a/leveler/commands/lvladmin/badge.py +++ b/leveler/commands/lvladmin/badge.py @@ -123,9 +123,9 @@ async def addbadge( badge_name = "{}_{}".format(name, serverid) if badge_name in userbadges.keys(): user_priority_num = userbadges[badge_name]["priority_num"] - new_badge[ - "priority_num" - ] = user_priority_num # maintain old priority number set by user + new_badge["priority_num"] = ( + user_priority_num # maintain old priority number set by user + ) userbadges[badge_name] = new_badge await self.db.users.update_one( {"user_id": user["user_id"]}, diff --git a/leveler/commands/lvladmin/debugging.py b/leveler/commands/lvladmin/debugging.py index 8e823ada..ad4cb614 100644 --- a/leveler/commands/lvladmin/debugging.py +++ b/leveler/commands/lvladmin/debugging.py @@ -50,9 +50,11 @@ async def debug_info(self, ctx): ("DB lock locked", self._db_lock.locked()), ( "DB lock queue", - "N/A" - if self._db_lock._waiters is None - else len(self._db_lock._waiters), + ( + "N/A" + if self._db_lock._waiters is None + else len(self._db_lock._waiters) + ), ), ("pymongo version", pymongoversion), ("motor version", motorversion), diff --git a/leveler/commands/lvlset/badge.py b/leveler/commands/lvlset/badge.py index dcd6d2d4..6bcac17b 100644 --- a/leveler/commands/lvlset/badge.py +++ b/leveler/commands/lvlset/badge.py @@ -112,9 +112,9 @@ async def buy_badge(self, ctx, is_global: Optional[bool], *, name: str): return if badge_info["price"] <= await bank.get_balance(user): await bank.withdraw_credits(user, badge_info["price"]) - userinfo["badges"][ - "{}_{}".format(name, str(serverid)) - ] = server_badges[name] + userinfo["badges"]["{}_{}".format(name, str(serverid))] = ( + server_badges[name] + ) await self.db.users.update_one( {"user_id": userinfo["user_id"]}, {"$set": {"badges": userinfo["badges"]}}, diff --git a/leveler/commands/top.py b/leveler/commands/top.py index 9a930a6b..1d9fbb14 100644 --- a/leveler/commands/top.py +++ b/leveler/commands/top.py @@ -64,8 +64,8 @@ async def top( elif options.global_top and owner: is_level = True if await self.config.global_levels() else False title = "Global Exp Leaderboard for {}\n".format(self.bot.user.name) - async for userinfo in self.db.users.find({}).allow_disk_use(True).sort( - "total_exp", -1 + async for userinfo in ( + self.db.users.find({}).allow_disk_use(True).sort("total_exp", -1) ): pos += 1 if is_level: @@ -109,9 +109,11 @@ async def top( icon_url = self.bot.user.avatar_url elif options.rep: title = "Rep Leaderboard for {}\n".format(server.name) - async for userinfo in self.db.users.find( - {f"servers.{server.id}": {"$exists": True}} - ).allow_disk_use(True).sort("rep", -1): + async for userinfo in ( + self.db.users.find({f"servers.{server.id}": {"$exists": True}}) + .allow_disk_use(True) + .sort("rep", -1) + ): pos += 1 users.append( ( @@ -137,10 +139,15 @@ async def top( else: is_level = True title = "Exp Leaderboard for {}\n".format(server.name) - async for userinfo in self.db.users.find( - {f"servers.{server.id}": {"$exists": True}} - ).allow_disk_use(True).sort( - [(f"servers.{server.id}.level", -1), (f"servers.{server.id}.current_exp", -1)] + async for userinfo in ( + self.db.users.find({f"servers.{server.id}": {"$exists": True}}) + .allow_disk_use(True) + .sort( + [ + (f"servers.{server.id}.level", -1), + (f"servers.{server.id}.current_exp", -1), + ] + ) ): pos += 1 if str(user.id) == userinfo["user_id"]: diff --git a/leveler/exp.py b/leveler/exp.py index 5ba8fd0d..f2277461 100644 --- a/leveler/exp.py +++ b/leveler/exp.py @@ -206,10 +206,10 @@ async def _handle_levelup(self, user, userinfo, server, channel): async def _find_server_rank(self, user, server): rank = 1 - async for userinfo in self.db.users.find( - {f"servers.{server.id}": {"$exists": True}} - ).allow_disk_use(True).sort( - [(f"servers.{server.id}.level", -1), (f"servers.{server.id}.current_exp", -1)] + async for userinfo in ( + self.db.users.find({f"servers.{server.id}": {"$exists": True}}) + .allow_disk_use(True) + .sort([(f"servers.{server.id}.level", -1), (f"servers.{server.id}.current_exp", -1)]) ): if userinfo["user_id"] == str(user.id): return rank @@ -217,9 +217,11 @@ async def _find_server_rank(self, user, server): async def _find_server_rep_rank(self, user, server): rank = 1 - async for userinfo in self.db.users.find( - {f"servers.{server.id}": {"$exists": True}} - ).allow_disk_use(True).sort("rep", -1): + async for userinfo in ( + self.db.users.find({f"servers.{server.id}": {"$exists": True}}) + .allow_disk_use(True) + .sort("rep", -1) + ): if userinfo.get("user_id") == str(user.id): return rank rank += 1 diff --git a/leveler/menus/top.py b/leveler/menus/top.py index 68e65db6..cda07c6d 100644 --- a/leveler/menus/top.py +++ b/leveler/menus/top.py @@ -115,9 +115,11 @@ def __init__( async def format_page(self, menu: TopMenu, entries): table = tabulate( entries, - headers=["#", self.board_type, "Level", "Username"] - if self.is_level - else ["#", self.board_type, "Username"], + headers=( + ["#", self.board_type, "Level", "Username"] + if self.is_level + else ["#", self.board_type, "Username"] + ), tablefmt="rst", ) table_width = len(table.splitlines()[0]) diff --git a/minecraftdata/minecraftdata.py b/minecraftdata/minecraftdata.py index 56e95f71..52367b3f 100644 --- a/minecraftdata/minecraftdata.py +++ b/minecraftdata/minecraftdata.py @@ -304,18 +304,20 @@ async def java(self, ctx, query_data: Optional[bool], server_ip: str): name=_("Players"), value="{0.players.online}/{0.players.max}\n{1}".format( status, - chat.box( - list( - chat.pagify( - await self.clear_mcformatting( - "\n".join([p.name for p in status.players.sample]) - ), - page_length=992, - ) - )[0] - ) - if status.players.sample - else "", + ( + chat.box( + list( + chat.pagify( + await self.clear_mcformatting( + "\n".join([p.name for p in status.players.sample]) + ), + page_length=992, + ) + )[0] + ) + if status.players.sample + else "" + ), ), ) embed.add_field( diff --git a/steamcommunity/steamcommunity.py b/steamcommunity/steamcommunity.py index 7026e189..c36b77e1 100644 --- a/steamcommunity/steamcommunity.py +++ b/steamcommunity/steamcommunity.py @@ -136,9 +136,11 @@ async def steamprofile(self, ctx, profile: SteamUser): title=profile.personaname, description=profile.personastate(), url=profile.profileurl, - timestamp=datetime.utcfromtimestamp(profile.lastlogoff) - if profile.lastlogoff - else discord.Embed.Empty, + timestamp=( + datetime.utcfromtimestamp(profile.lastlogoff) + if profile.lastlogoff + else discord.Embed.Empty + ), color=profile.personastatecolor, ) if profile.gameid: