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

Option to include number of very active reviewers in report #114

Open
MaineC opened this issue Sep 11, 2023 · 3 comments
Open

Option to include number of very active reviewers in report #114

MaineC opened this issue Sep 11, 2023 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers keep

Comments

@MaineC
Copy link
Contributor

MaineC commented Sep 11, 2023

One metric that we found helpful at $dayjob was looking at how many people there are that provide the bulk of mentorship for contributors. If that drops below a certain line (2-3 people minimum) it typically means the project is in danger.

Technically what we do is simple: Compute the number of people that provide at least $n comments over time period $x.

Is there any way to retrieve that kind of information in a/this GitHub action?

@zkoppert zkoppert added the enhancement New feature or request label Sep 11, 2023
@zkoppert
Copy link
Member

Really interesting metric to look at for a project! Thanks for the recommendation.

I believe it is possible to pull usernames for commits/comments and the associated timestamp with the GitHub API although I imagine this would take a long time for large projects as the GitHub action would need to pull/store/analyze each comment for every issue and pr in the last x time.

We could put that into this GitHub action and if the performance impact is heavy, we could just not have the option enabled by default.

@zkoppert zkoppert added the good first issue Good for newcomers label Sep 11, 2023
@MaineC
Copy link
Contributor Author

MaineC commented Sep 13, 2023

One thing that just came to my mind that supports the idea to make this a non-default option:

The "are there enough people around to consider this project alive" in my experience only arises for relatively stable, quiet projects anyway - both, in InnerSource and Open Source - think " Should this project be moved to https://attic.apache.org/?" type of questions.

For larger, more busy ones I would expect the "time to first response" etc. to give a better first overview. If there is a risk for the project to fall below a sustainable number of reviewers one should see those times go up pretty quickly. For the "go up" part though one would need to run this action regularly and create a time series out of the results.

Copy link

This issue is stale because it has been open 21 days with no activity. Remove stale label or comment or this will be closed in 14 days.

@github-actions github-actions bot added the Stale label Mar 14, 2024
@zkoppert zkoppert added keep and removed Stale labels Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers keep
Projects
None yet
Development

No branches or pull requests

2 participants