Skip to content

Conversation

@onerandomusername
Copy link
Contributor

closes GH-137

@python-discord-policy-bot python-discord-policy-bot bot requested a review from a team November 11, 2025 21:24
Copy link
Member

@decorator-factory decorator-factory left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor nitpicks

aoc_name = aoc_name[1:-1]

# Check if an advent of code account is linked in the Redis Cache if aoc_name is not given
is_explict_name: bool = bool(aoc_name)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why the annotation?

if (aoc_cache_name := await _caches.account_links.get(ctx.author.id)) and aoc_name is None:
aoc_name = aoc_cache_name

name_not_found: bool = False
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

for rank, data in enumerate(leaderboard.values(), start=1):
if self_placement_name and data["name"].lower() == self_placement_name.lower():
if use_you_for_placement:
name = f"(You) {data['name']}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's use a consistent quote style. Since Python 3.12 you can do f"(You) {data["name"]}"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AoC: If linked username is not shown in leaderboard, include link to the regular leaderboard

2 participants