[Koha-devel] Follow-up patches and why not to use them

Marcel de Rooy rooy.de.m at gmail.com
Fri Jun 18 09:40:47 CEST 2021


Yeah I probably missed it while catching up ;)
What you say makes sense, but the problem probably boils down to the gap
between theory and practice.

Op vr 18 jun. 2021 om 09:36 schreef Joonas Kylmälä <
joonas.kylmala at helsinki.fi>:

> Hi Marcel,
>
> On 18/06/2021 09:45, Marcel de Rooy wrote:
> > Agree with Julian here. It depends.
> > You could squash follow-up patches yourself too before reverting them?
> > In some cases patches tell us a nice story, in a lot of cases it might
> > be confusing or messy..
>
> I think you might not have seen yet my follow-up email yet to Julian
> sent on the 7th of June. Basically squashing or reverting blindly is not
> an option when you want to review that the changes make sense on top of
> the new code base. Then the "nice story" or "confusing/messy story"
> part: as I said in the follow-up email to Julian my wish was to have as
> well commits with one logical change in each so then we should have
> *always* a nice story to read.
>
> Would love to hear your opinion now with these points clarified.
>
> Joonas
> _______________________________________________
> Koha-devel mailing list
> Koha-devel at lists.koha-community.org
> https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
> website : https://www.koha-community.org/
> git : https://git.koha-community.org/
> bugs : https://bugs.koha-community.org/
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.koha-community.org/pipermail/koha-devel/attachments/20210618/34caaa89/attachment.htm>


More information about the Koha-devel mailing list