Skip to content

add "Total Contributions" stat - #482

Open
martin-mfg wants to merge 17 commits into
masterfrom
contributions-stat
Open

add "Total Contributions" stat#482
martin-mfg wants to merge 17 commits into
masterfrom
contributions-stat

Conversation

@martin-mfg

@martin-mfg martin-mfg commented Aug 15, 2026

Copy link
Copy Markdown
Member

Add a new optional stat for the stats card, "Total Contributions".

  • I think this is something many users want. There are many issues where people wonder why their "contributed to" or their "total commits" doesn't match their contributions.
  • This PR also introduces a way to fetch and sum stats which are only available for 1 year at a time in the GitHub API. We could reuse this later for more highly requested all-time stats if necessary.

I will add documentation in a separate PR.

@vercel

vercel Bot commented Aug 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
github-stats-extended-backend Ready Ready Preview Aug 15, 2026 3:38pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
github-stats-extended-frontend Ignored Ignored Aug 15, 2026 3:38pm

Request Review

@martin-mfg
martin-mfg requested a review from marcalexiei August 15, 2026 15:38

@marcalexiei marcalexiei left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Opened #484 with few followup improvements

Comment on lines +620 to +637
const stats = await fetchStats(
"anuraghazra",
false,
[],
false,
false,
false,
undefined,
[],
[],
false,
false,
false,
false,
false,
[],
true, // include_contributions
);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking at this, I’m wondering if we should change the function arguments to an options object 😅.

Right now, only fetchWakatimeStats follows this approach.
I would consider moving all the fetchers to this format in a separate PR.

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.

2 participants