Skip to content

Tied developers share a place in rankings - #102

Closed
vbarhatov wants to merge 2 commits into
mainfrom
ranking-ties-share-place
Closed

vbarhatov wants to merge 2 commits into
mainfrom
ranking-ties-share-place

Conversation

@vbarhatov

@vbarhatov vbarhatov commented Oct 21, 2026 •

Copy link
Copy Markdown

duplicates PR #103

vbarhatov and others added 2 commits October 21, 2026 19:51
Equal impact now gets the same place and the next one skips ahead
(1, 1, 1, 4) instead of i + 1. Applies to the developer table, org
page, spend table, profile, CSV/Sheets export, AI summary and
highlights. Row order is unchanged.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Teams are ordered by the sort column, then impact (A), then name. The
place uses (sort column, impact (A)) as its key, so only teams equal on
both share a place; a hidden spend value never ties.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@vbarhatov
vbarhatov requested a review from msogin October 21, 2026 18:04
@vbarhatov vbarhatov closed this Oct 21, 2026
@vbarhatov vbarhatov changed the title Tied developers and teams share a place in rankings Tied developers share a place in rankings Oct 21, 2026
@vbarhatov
vbarhatov removed the request for review from msogin October 21, 2026 18:17
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