3 ms·
Here's some insights for PRs created within the last 120 days for the mentioned repos. 88 PRs created: https://reviews.gitsense.com/insights/github?p=days-open
by sdesol 3y ago
Here's some insights for PRs created within the last 120 days for the mentioned repos.
88 PRs created: https://reviews.gitsense.com/insights/github?p=days-open&q=author%3Agitstart%2Bauthor%3Agitstart-calcom%2Bauthor%3Agitstart-sourcegraph%2Bauthor%3Agitstart-twenty%2Bpull-age%3A%3C%3D120&t=crc-insights&tb=all&v=calcom%2Fcal.com%20----%20sourcegraph%2Fsourcegraph%20----%20storybookjs%2Fstorybook%20----%20strapi%2Fstrapi%20----%20supabase%2Fsupabase%20----%20twentyhq%2Ftwenty%3A%3Agithub%3Acalcom%2Fcal.com%2Cgithub%3Asourcegraph%2Fsourcegraph%2Cgithub%3Astorybookjs%2Fstorybook%2Cgithub%3Astrapi%2Fstrapi%2Cgithub%3Asupabase%2Fsupabase%2Cgithub%3Atwentyhq%2Ftwenty%3A%3A%3A%3A%3A%3A1691426260099 https://reviews.gitsense.com/insights/github?p=days-open&q=a...
70 merged: https://reviews.gitsense.com/insights/github?p=days-open&q=author%3Agitstart%2Bauthor%3Agitstart-calcom%2Bauthor%3Agitstart-sourcegraph%2Bauthor%3Agitstart-twenty%2Bmerged%3Atrue%2Bpull-age%3A%3C%3D120&t=crc-insights&tb=closed&v=calcom%2Fcal.com%20----%20sourcegraph%2Fsourcegraph%20----%20storybookjs%2Fstorybook%20----%20strapi%2Fstrapi%20----%20supabase%2Fsupabase%20----%20twentyhq%2Ftwenty%3A%3Agithub%3Acalcom%2Fcal.com%2Cgithub%3Asourcegraph%2Fsourcegraph%2Cgithub%3Astorybookjs%2Fstorybook%2Cgithub%3Astrapi%2Fstrapi%2Cgithub%3Asupabase%2Fsupabase%2Cgithub%3Atwentyhq%2Ftwenty%3A%3A%3A%3A%3A%3A1691426260099 https://reviews.gitsense.com/insights/github?p=days-open&q=a...
7 rejected: (closed/not merged) https://reviews.gitsense.com/insights/github?p=days-open&q=author%3Agitstart%2Bauthor%3Agitstart-calcom%2Bauthor%3Agitstart-sourcegraph%2Bauthor%3Agitstart-twenty%2Bpull-age%3A%3C%3D120%2Brejected%3Atrue&t=crc-insights&tb=closed&v=calcom%2Fcal.com%20----%20sourcegraph%2Fsourcegraph%20----%20storybookjs%2Fstorybook%20----%20strapi%2Fstrapi%20----%20supabase%2Fsupabase%20----%20twentyhq%2Ftwenty%3A%3Agithub%3Acalcom%2Fcal.com%2Cgithub%3Asourcegraph%2Fsourcegraph%2Cgithub%3Astorybookjs%2Fstorybook%2Cgithub%3Astrapi%2Fstrapi%2Cgithub%3Asupabase%2Fsupabase%2Cgithub%3Atwentyhq%2Ftwenty%3A%3A%3A%3A%3A%3A1691426260099 https://reviews.gitsense.com/insights/github?p=days-open&q=a...
Interesting take away is, the merge rate is quite high, but since we can assume these PRs were created for a reason, can we also say 7 not being merged is also bad?
Full Disclosure: This is my tool
- hzia 3y agoLove the analysis! Usually the closed number is higher because there are still many edge cases where PRs get closed outside of bad code. E.g bug already fixed in master, follow up PR, PRs merging into a larger one and so on I think basic NLP on the PR decision may be able to automatically differentiate them
- sdesol 3y agoThanks for the info. Would you say PRs that have been open for a long time (28+ days with no activity) is due to priority changes? https://reviews.gitsense.com/insights/github?p=days-open&q=author%3Agitstart%2Bauthor%3Agitstart-calcom%2Bauthor%3Agitstart-sourcegraph%2Bauthor%3Agitstart-twenty%2Bpull-age%3A%3C%3D120%2Bready%3Atrue%2Bdays-open%3A%3E%3D28&t=crc-insights&tb=open&v=calcom%2Fcal.com%20----%20sourcegraph%2Fsourcegraph%20----%20storybookjs%2Fstorybook%20----%20strapi%2Fstrapi%20----%20supabase%2Fsupabase%20----%20twentyhq%2Ftwenty%3A%3Agithub%3Acalcom%2Fcal.com%2Cgithub%3Asourcegraph%2Fsourcegraph%2Cgithub%3Astorybookjs%2Fstorybook%2Cgithub%3Astrapi%2Fstrapi%2Cgithub%3Asupabase%2Fsupabase%2Cgithub%3Atwentyhq%2Ftwenty%3A%3A%3A%3A%3A%3A1691426260099 https://reviews.gitsense.com/insights/github?p=days-open&q=a...
- hzia 3y agoQuite a few are due to priority changes over time. We are thinking to automatically close PRs when there is no activity for > 14 days.