Skip to content

fix: make non-top ranks explicit - #124

Open
Victorakinwunmi wants to merge 2 commits into
SPulse-Org:mainfrom
Victorakinwunmi:fix/explicit-top-rank
Open

fix: make non-top ranks explicit#124
Victorakinwunmi wants to merge 2 commits into
SPulse-Org:mainfrom
Victorakinwunmi:fix/explicit-top-rank

Conversation

@Victorakinwunmi

Copy link
Copy Markdown

Summary

  • return Some(rank) only for players in the tracked top-50
  • return None for players outside the top-50 instead of the ambiguous rank

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.

3 participants