3 ms·
Love 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 i
by hzia 3y ago
Love 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.