Skip to content

Use direct avatars.githubusercontent.com URL for contributor avatars #24

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
Copilot wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Use direct avatars.githubusercontent.com URL for contributor avatars #24

Changes from 3 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
f5a537c
Add contributor avatars above username in leaderboard tables
Copilot Apr 27, 2026
10ab2bf
Use avatars.githubusercontent.com URL to avoid redirect
Copilot Apr 27, 2026
88b4073
Replace mixed HTML/Markdown contributor cells with pure HTML anchor tags
Copilot Apr 27, 2026
6524835
Add alt attribute, ?s=64 to avatar URLs, and extract _contributor_cel…
Copilot Apr 27, 2026
ca23512
Validate, URL-encode, and HTML-escape login in _contributor_cell()
Copilot Apr 27, 2026
c945a26
Tighten _GITHUB_LOGIN_RE to reject consecutive hyphens
Copilot Apr 27, 2026
cf61618
Handle invalid logins gracefully in _contributor_cell()
Copilot Apr 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 12 additions & 2 deletions scripts/leaderboard.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -767,8 +767,13 @@ def generate_markdown(contributors, levels_data):
commits_display += f" · 🤝 {coauthored}"
commits_display += f" · 📦 {repos_count}"

contributor_cell = (
f'<a href="https://github.com/{login}">'
f'<img src="https://avatars.githubusercontent.com/{login}" width="32" height="32"><br>'
Comment thread
jbampton marked this conversation as resolved.
Outdated
Comment thread
jbampton marked this conversation as resolved.
Outdated
f"@{login}</a>"
)
Comment thread
jbampton marked this conversation as resolved.
Outdated
lines.append(
f"| {rank} | [@{login}](https://github.com/{login})"
f"| {rank} | {contributor_cell}"
f" | {level} | {rarity_display} | {commits_display}"
f" | {prog} | {streak_display}"
f" | {badges} | {points_display} |"
Expand Down Expand Up @@ -812,8 +817,13 @@ def generate_markdown(contributors, levels_data):
breakdown_parts.append(f"📁 {other_c}")
breakdown = " · ".join(breakdown_parts) if breakdown_parts else "—"

contributor_cell = (
f'<a href="https://github.com/{login}">'
f'<img src="https://avatars.githubusercontent.com/{login}" width="32" height="32"><br>'
Comment thread
jbampton marked this conversation as resolved.
Outdated
Comment thread
jbampton marked this conversation as resolved.
Outdated
f"@{login}</a>"
)
Comment thread
jbampton marked this conversation as resolved.
Outdated
Comment thread
jbampton marked this conversation as resolved.
Outdated
lines.append(
f"| {i} | [@{login}](https://github.com/{login})"
f"| {i} | {contributor_cell}"
f" | {first_date} | {last_date}"
f" | {days_active} | {cpd}"
f" | {breakdown} | Top {pctile}% |"
Expand Down