<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le mar. 7 mars 2023 à 00:15, AnnoyingRains <<a href="mailto:annoyingrain5@gmail.com">annoyingrain5@gmail.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> We should never close a MR automatically. Only a maintainer of a project or the author itself should close a MR.<br>
<br>
I agree with not closing MRs automatically. As I stated in my original<br>
message, we are no longer planning on doing that, it's not helpful and<br>
is only destructive.<br>
The decision to close an MR will be left with the specific project's<br>
maintainers and the person who opened the MR.<br>
<br>
> The decision if a MR should be closed or not is often depending on a context (e.g. a MR required another MR to be merged first<br>
> and it is taking more time than expected, the author is busy with other things but will eventually come back to it, ...)<br>
> and a script is unable to see this.<br>
<br>
I would also argue that humans that are not a maintainer of that<br>
specific project should not close an MR for similar reasons. There is<br>
a lot here that the gardening team would need to know about every<br>
project<br>
<br>
> for GCompris, we don't have a lot of MR and even if some are old, we still plan to take over them at some point (we know which ones are unmaintained) so I think it's preferable to not add messages.<br>
<br>
We can exclude Gcompris if you feel it is needed, but I am unsure how<br>
labeling MRs as stale and reminding authors wouldn't be preferable.<br>
<br></blockquote><div><br></div><div>Hi,</div><div><br></div><div>It will be considered as noise and ignored. We have two use cases:</div><div>* Either the person is following and if a MR is stalling it's because maintainers don't have enough time to handle it now.</div><div>* Either the person is no more following (for multiple reasons) and maintainers already know it and they plan to handle the MR later.</div><div><br></div><div>In both cases, no action will be taken.</div><div><br></div><div>As there are not a lot of MR (6), it's an amount we can handle it by ourselves.</div><div>I think what you propose is more important and has more values when there a lot of MR opened and it's more complicated to handle them (or more maintainers).²<br></div><div><br></div><div>Cheers,</div><div><br></div><div>Johnny<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> but this should probably be done manually<br>
<br>
We are planning on doing this manually until all of the issues have<br>
been ironed out perfectly and we know a foolproof way to ensure<br>
nothing is ever poked that shouldn't be, which may never happen.<br>
We will open another discussion before automating anything, due to the<br>
potential disaster a bug could cause.<br>
<br>
> "Hi, I really like this work, do you intent to continue working<br>
> on it or can I take over" than a generic message that feels fake.<br>
<br>
This is great, but not exactly what this inititive is about.<br>
This is more for reminding people that the MR exists (even had one<br>
case of "oh! I forgot I made this MR" in my small scale test!), and<br>
labeling MRs so that contributors feel more free to request taking<br>
over the project.<br>
Basically, we cannot really take over every stale MR in the entirety of KDE.<br>
<br>
- Kye Potter, KDE Gardening<br>
<br>
<br>
On Tue, Mar 7, 2023 at 9:39 AM Albert Astals Cid <<a href="mailto:aacid@kde.org" target="_blank">aacid@kde.org</a>> wrote:<br>
><br>
> El dilluns, 6 de març de 2023, a les 15:18:42 (CET), Carl Schwan va escriure:<br>
> > Hi,<br>
> ><br>
> > We should never close a MR automatically. Only a maintainer of a project<br>
> > or the author itself should close a MR. The decision if a MR should be<br>
> > closed or not is often depending on a context (e.g. a MR required another<br>
> > MR to be merged first and it is taking more time than expected, the<br>
> > author is busy with other things but will eventually come back to it, ...)<br>
> > and a script is unable to see this.<br>
> ><br>
> > I do not mind poking people semi-regularly (every 6 months or so), but again<br>
> > this should probably be done manually and with a small personalized message<br>
> > for example: "Hi, I really like this work, do you intent to continue<br>
> > working on it or can I take over" than a generic message that feels fake.<br>
> ><br>
> > I really hate communicating with robots instead of with humans and I really<br>
> > see the trends of trying to automatize all our interaction with dumb robots<br>
> > and chat bots in our society really worrying.<br>
> ><br>
> > If we want to automatize, we should instead try to list the stale MRs and<br>
> > maybe send the list to the mailing list once a month so that people are<br>
> > reminded about them and take a look at which one they may be able to unlock.<br>
><br>
> We already have that, they get posted to<br>
>  <a href="https://invent.kde.org/teams/gardening/gitlab/-/issues" rel="noreferrer" target="_blank">https://invent.kde.org/teams/gardening/gitlab/-/issues</a><br>
> weekly.<br>
><br>
> What i forgot is what i did to be notified of it by email ^_^<br>
><br>
> Cheers,<br>
>   Albert<br>
><br>
> ><br>
> > Cheers,<br>
> > Carl<br>
> ><br>
> ><br>
> > ------- Original Message -------<br>
> ><br>
> > Le dimanche 5 mars 2023 à 11:13 AM, AnnoyingRains <<a href="mailto:annoyingrain5@gmail.com" target="_blank">annoyingrain5@gmail.com</a>><br>
> a écrit :<br>
> > > For a short amount of time now, there have been some small-scale<br>
> > > trials of replying to old MRs with a reminder, and suggesting that the<br>
> > > author closes the MR if it is either no longer needed or if it needs<br>
> > > more work and the author does not have time for it.<br>
> > ><br>
> > > This has appeared to have a positive impact on the state of KDE<br>
> > > software from this (albeit limited) trial. Some MRs have had renewed<br>
> > > interest, others have admitted that they had forgotten that the MR<br>
> > > even existed.<br>
> > ><br>
> > > We did consider closing MRs if there was no activity after our ping<br>
> > > message. We are no longer planning on doing this, as it is more<br>
> > > destructive than helpful. All decisions on if a MR should be closed<br>
> > > will be left with the maintainers and the person who opened the MR.<br>
> > ><br>
> > > So, we need a proper discussion about this, should we send these<br>
> > > reminder messages at all? If so, how old should an MR be before<br>
> > > sending this reminder? Should closing the MR even be suggested in the<br>
> > > message?<br>
> > ><br>
> > > If your specific project does not play nicely with this programme,<br>
> > > please let us know and we can add it to the list of exclusions on our<br>
> > > KDE Community page.<br>
> > ><br>
> > > I need your input,<br>
> > > - Kye Potter, KDE Gardening<br>
><br>
><br>
><br>
><br>
</blockquote></div></div>