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@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@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/