Skip to content
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

Scrape reviewer count and publish on website #101

Draft
wants to merge 23 commits into
base: main
Choose a base branch
from

Conversation

GeckoEidechse
Copy link
Member

@GeckoEidechse GeckoEidechse commented Aug 26, 2024

The idea behind this is to showcase the people putting in the work to review and give feedback on PRs.

TODOs:

  • Clean up script (partially LLM generated but it works lol)
  • Showcase review count on website
  • Add CI logic to periodically call the script
  • Expand to also track
    • weekly stats
    • monthly stats
      (as it's near impossible for someone completely new to the project to catch up otherwise
  • Filter out "LGTM" and similar types of reviews that encourage review spam
  • ???

@GeckoEidechse GeckoEidechse marked this pull request as draft August 26, 2024 14:05
GeckoEidechse and others added 22 commits October 4, 2024 10:53
The previous check was wrong and ignored all top level reviews, meaning it only counted comments on reviews
Adds an initial function to filter out reviews like "LGTM" that are not helpful in deciding whether a PR should be approved or not
stats should be added by separate commit from CI later
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Gecko Doing
Development

Successfully merging this pull request may close these issues.

1 participant