Skip to content

Resolve Ambiguity for ties in Sort by Groups #2376

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.

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

Open
joeng03 opened this issue Apr 3, 2025 · 0 comments
Open

Resolve Ambiguity for ties in Sort by Groups #2376

joeng03 opened this issue Apr 3, 2025 · 0 comments

Comments

@joeng03
Copy link
Contributor

joeng03 commented Apr 3, 2025

Related to #2331

Several sorting options face the issue where there is no defined order if the sort values have a tie, for example, when grouped by repo, sort the groups by author displayName (there is a possible case where an author contributed to multiple repos, and is the lexicographically largest display name among contributors of both repos.

I would propose to resolve this ambiguity in a PR, where each grouping option (by author or repo) can only sort by a fixed set of fields (for example, sort by default order might not make sense for group by author, and sort by author displayName might not make sense for group by repo). Then for sort fields that might have the same value (for example, totalCommits might be the same for several authors/repos), maybe define a default way to break ties by unique fields (for example, the default ordering of repos, or lexicogaphical order of author GitId.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

1 participant