*committers (and not reviewers)

On Mon, Apr 14, 2025 at 12:13 PM Paulo Motta <pa...@apache.org> wrote:

> >  I am not sure why it is so hard for people to not forget to close a PR
> when their branch is merged.
>
> I wonder if reviewers* know they need to append the message "Closes
> #PR_ID" to the end of the commit message to have the PR be closed, this
> does not seem very obvious, but it's also a bit inconvenient.
>
> Since Apache INFRA already links github PRs to the appropriate JIRA, it
> would probably not be very hard to have the PR be closed when the JIRA is
> resolved. If this does not sound stupid perhaps we could submit an INFRA
> feature request to address this issue.
>
> On Mon, Apr 14, 2025 at 3:17 AM Štefan Miklošovič <smikloso...@apache.org>
> wrote:
>
>> I am not sure why it is so hard for people to not forget to close a PR
>> when their branch is merged. I stopped "fighting" this and I just run a
>> script every few weeks. Funny that people don't forget to create a PR when
>> trying to make a change but as soon as it is delivered the respective PR is
>> "memory holed". A PR does not close itself if it was not obvious already.
>>
>> On Mon, Apr 14, 2025 at 8:00 AM Bernardo Botella <
>> conta...@bernardobotella.com> wrote:
>>
>>> Thanks Josh and Stefan for the comments!
>>>
>>> Such a script can definitely be helpful for this purpose of keeping our
>>> house tidy. It seems that the thread hasn’t gotten much steam yet. As this
>>> is, by no means, any urgent matter, let’s give some more time for people to
>>> pitch in. I’ll wait some more days looking for answers on this thread.
>>> Then, if no one has any strong opinion against it, I can start closing old
>>> PRs.
>>>
>>> Thanks!
>>> Bernardo
>>>
>>> On Apr 11, 2025, at 10:22 AM, Štefan Miklošovič <smikloso...@apache.org>
>>> wrote:
>>>
>>> I have a small script which scans GH pull requests (their titles) and
>>> looks into JIRA to see what is their status. When it is "resolved" it
>>> prints it to the console. Then I go over the links of PRs and close them
>>> one by one. This relies on the title of the PR to be in exact format
>>> (CASSANDRA-123 a title of the ticket) and not bullet proof but I have not
>>> come up with anything better so far.
>>>
>>> On Fri, Apr 11, 2025 at 5:19 PM Josh McKenzie <jmcken...@apache.org>
>>> wrote:
>>>
>>>> +1 from me.
>>>>
>>>> My intuition is that this is a logical consequence of us not using
>>>> github to merge PR's so they don't auto-close. Which seems like it's a
>>>> logical consequence of us using merge commits instead of per-branch commits
>>>> of patches.
>>>>
>>>> The band-aid of at least having a human-in-the-loop to close out old
>>>> inactive things is better than the status quo; the information is all still
>>>> available in github but the status of the PR's will communicate different
>>>> things.
>>>>
>>>> On Thu, Apr 10, 2025, at 7:14 PM, Bernardo Botella wrote:
>>>>
>>>> Hi everyone!
>>>>
>>>> First of all, this may have come out before, and I understand it is
>>>> really hard to keep a tidy house with so many different collaborations.
>>>> But, I can't help the feeling that coming to the main Apache Cassandra
>>>> repository and seeing more than 600 open PRs, some of them without activity
>>>> for 5+ years, gives the wrong impression about the love and care that we
>>>> all share for this code base. I think we can find an easy to follow
>>>> agreement to try and keep things a bit tidier. I wanted to propose some
>>>> kind of "rule" that allow us to directly close PRs that haven't had
>>>> activity in a reasonable and conservative amount of time of, let's say, 6
>>>> months? I want to reiterate that I mean no activity at all for six months
>>>> from the PR author. I understand that complex PRs can be opened for longer
>>>> than that period, and that's perfectly fine.
>>>>
>>>> What do you all think?
>>>>
>>>> Bernardo
>>>>
>>>>
>>>>
>>>

Reply via email to