Skip to content

Show LOC for public solutions#248

Merged
msaroufim merged 1 commit into
mainfrom
loc-public-solution-lines
Jul 5, 2026
Merged

Show LOC for public solutions#248
msaroufim merged 1 commit into
mainfrom
loc-public-solution-lines

Conversation

@msaroufim

@msaroufim msaroufim commented Jul 5, 2026

Copy link
Copy Markdown
Member

Summary

  • Add optional line_count to /api/leaderboard ranking rows only when the leaderboard is concluded and public solution code is allowed.
  • Show LOC directly in each ranking row when line_count is present, next to the existing ranking metadata.

Validation

  • git diff --check
  • .venv/bin/ruff check kernelboard/api/leaderboard.py
  • npm run lint
  • npm run build

Notes

  • Backend route pytest still requires Docker for the Postgres/Redis fixtures; Docker is not running in this local environment.

@msaroufim msaroufim marked this pull request as ready for review July 5, 2026 02:31
@msaroufim msaroufim merged commit 0658af5 into main Jul 5, 2026
1 check passed
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.

1 participant