You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Badge Request: Total commits of GitHub user #11130
I think this request could appear like it has some artificial boundaries, so I wanted to add that some elements (like the seemingly magic 200 number) are ostensibly being mentioned here because those were set previously for another GitHub badge (refs #5507) and so would likely be reasonable limits to include here as well
I suspect that using the v3 rest api would probably be suboptimal here as that would require quite a large number of requests to be issued in total, and several strings of those calls would have to be sequenced together (e.g. get all repos, get stars for each repo, get commits for top 200 most starred repos).
I'd instead suggest reviewing GitHub's v4 api (graph based) and determining whether the necessary data could be retrieved with a single query, or at least significantly fewer queries than would be required with v3/rest
Activity
calebcartwright commentedon Jun 11, 2025
I think this request could appear like it has some artificial boundaries, so I wanted to add that some elements (like the seemingly magic 200 number) are ostensibly being mentioned here because those were set previously for another GitHub badge (refs #5507) and so would likely be reasonable limits to include here as well
I suspect that using the v3 rest api would probably be suboptimal here as that would require quite a large number of requests to be issued in total, and several strings of those calls would have to be sequenced together (e.g. get all repos, get stars for each repo, get commits for top 200 most starred repos).
I'd instead suggest reviewing GitHub's v4 api (graph based) and determining whether the necessary data could be retrieved with a single query, or at least significantly fewer queries than would be required with v3/rest