Petr, Any suggestions on how to fix this? One of the ideas that come to my mind is to unify the process. For example, by enforcing that all changes are merged only through PRs. That, however, means that all contributors are forced to work with GitHub, which is not necessarily a great thing.
Thoughts? -Val On Wed, Feb 24, 2021 at 12:45 AM Petr Ivanov <mr.wei...@gmail.com> wrote: > I'd guess that one of the main problem with inactive PRs is in creation of > PR for reviewing but merging it from command line (not via GitHub > interface). > Also, of course, there are lots of efforts which are abandoned after first > review, or even do not have a chance to be reviewed at all. > > > > On 22 Feb 2021, at 11:51, Stephen Darlington < > stephen.darling...@gridgain.com> wrote: > > > > I think we need to be able answer the question “Why are there so many > inactive PRs?" before we automate their removal. If perfectly good changes > are being ignored, we have a problem. > > > > Removing branches of merged PRs and protecting the main branch make > sense. > > > >> On 20 Feb 2021, at 18:30, Pavel Tupitsyn <ptupit...@apache.org> wrote: > >> > >> +1 > >> > >> - Close inactive PRs (1 month or so?) > >> - Enable main branch protection (no force pushes, require linear > history, > >> require status checks) > >> > >> On Sat, Feb 20, 2021 at 2:31 PM Petr Ivanov <mr.wei...@gmail.com> > wrote: > >> > >>> Hi, Igniters! > >>> > >>> > >>> When we started Ignite 3.x in new repository, not only we have > received a > >>> chance to cleanup codebase, but to maintain some order in development > >>> tools, like GitHub. > >>> Currently in 2.x repository we have lots of stalled PRs and branches, > >>> which not only clog the repository, but also indirectly influence TC > >>> performance (due to necessity to check for updates every ref: branches > and > >>> PRs). > >>> > >>> Could I suggest we devise some recommendations for using PR's and > branches > >>> in new repo and add some rules about stalled PRs at least, like closing > >>> them if inactive for some time. > >>> Also we can activate some settings in repo's configuration, like auto > >>> delete branch after PR is merged. > >>> > >>> > >>> WDYT? > > > > > >
