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

[tools-public] Add KPI for PR reviews by team members for Core #223

Closed
wants to merge 6 commits into from

Conversation

mnajdova
Copy link
Member

@mnajdova mnajdova commented Oct 28, 2024

Added metric for a number of reviews from the Core team per repository.

Screenshot 2024-10-29 at 14 53 45

@mnajdova mnajdova marked this pull request as ready for review October 29, 2024 13:50
@mnajdova
Copy link
Member Author

I am not sure why the netlify deploy is skipped: https://app.netlify.com/sites/mui-dashboard/deploys/6720e80992fcaa0008a50f34

@mnajdova mnajdova changed the title [tools-public] Scaffold Core team page [tools-public] Add KPI for PR reviews by team members for Core Oct 29, 2024
@mnajdova
Copy link
Member Author

mnajdova commented Oct 30, 2024

I interpolated locally the data that is available from ossinsights, namely the line on the chart represents your + Sebastian's reviews on the mui/material-ui repo starting from 2020-08-01.

Screenshot 2024-10-30 at 14 17 15

The bars represent respectively mui/material, mui/base-ui and mui/pigment-css. The Toolpad Chart didn't have the stacking bar option, but I can create a custom component if that helps with getting a better picture.


Anyway, I restored back to the previous state, I think we can aim at 1000 reviews per month from the team, looks like a good KPI.

type = 'PullRequestReviewEvent' AND
action = 'created' AND
repo_id = ${repoMap[repo]} AND
actor_login IN ('mnajdova', 'siriwatknp', 'Janpot', 'DiegoAndai', 'mj12albert', 'aarongarciah', 'brijeshb42', 'michaldudak', 'colmtuite', 'atomiks', 'vladmoroz')
Copy link
Member

Choose a reason for hiding this comment

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

I think to replace with

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: tools Toolpad support apps
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants