apex-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Thomas Weise <...@apache.org>
Subject Re: [DISCUSS] inactive PR
Date Sun, 24 Sep 2017 16:21:42 GMT
On Sun, Sep 24, 2017 at 9:08 AM, Pramod Immaneni <pramod@datatorrent.com>
wrote:

>
> > On Sep 24, 2017, at 8:28 AM, Thomas Weise <thw@apache.org> wrote:
> >
> > +1 for closing inactive PRs after documented period of inactivity
> > (contributor guidelines)
> >
> > There is nothing "draconian" or negative about closing a PR, it is a
> > function that github provides that should be used to improve
> collaboration.
> > PR is a review tool, it is not good to have stale or abandoned PRs
> sitting
> > as open. When there is no activity on a PR and it is waiting for action
> by
> > the contributor (not ready for review), it should be closed and then
> > re-opened once the contributor was able to move it forward and it becomes
> > ready for review.
> >
> > Thomas
>
> Please refer to my email again, I am not against closing PR if there is
> inactivity. My issue is with the time period. In reality, most people will
> create new PRs instead of reopening old ones and the old context/comments
> will be forgotten and not addressed.
>
>
Why will contributors open new PRs even in cases where changes are
requested on an open PR? Because it is not documented or reviewers don't
encourage the proper process? We should solve that problem.

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message