Skip to content

feat: add Point Source Breakdown section to contribution overviews#278

Draft
RaghavArora14 wants to merge 1 commit intotscircuit:mainfrom
RaghavArora14:feature/point-source-breakdown
Draft

feat: add Point Source Breakdown section to contribution overviews#278
RaghavArora14 wants to merge 1 commit intotscircuit:mainfrom
RaghavArora14:feature/point-source-breakdown

Conversation

@RaghavArora14
Copy link
Contributor

wip

  • Add prPoints, reviewPoints, discussionPoints to ContributorScore interface
  • Track point sources in getContributorScore function
  • Generate Point Source Breakdown table in markdown showing:
    • PR Points, Review Points, Discussion Points
    • Total Score
    • Percentage breakdown (PR %, Review %, Discussion %)
  • Add test assertions for new section

- Add prPoints, reviewPoints, discussionPoints to ContributorScore interface
- Track point sources in getContributorScore function
- Generate Point Source Breakdown table in markdown showing:
  - PR Points, Review Points, Discussion Points
  - Total Score
  - Percentage breakdown (PR %, Review %, Discussion %)
- Add test assertions for new section
@vercel
Copy link

vercel bot commented Dec 8, 2025

@RaghavArora14 is attempting to deploy a commit to the tscircuit Team on Vercel.

A member of the Team first needs to authorize it.

@RaghavArora14 RaghavArora14 marked this pull request as draft December 8, 2025 09:33
result.score += reviewPoints
result.reviewPoints = reviewPoints

// Discussion points are currently 0 (not implemented yet)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not implemented???

Copy link
Contributor Author

Choose a reason for hiding this comment

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

about that i was a little confused, the current scoring system isnt accounting for discussion points right? thats sort of why i have set it to 0?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

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