Dear Stéfan, Since I just did *not* do it :s ... I realize that it could/should be the responsibility of the core dev who is squashing and merging to reword, if necessary, the 'main' commit message (which, if left unchanged, ends up being the PR title) and, perhaps(?), even the secondary commit messages (corresponding to the original individual commits). If anything, I would think that squash merges improve the situation, otherwise those 'long list[s] of items like "polish things up a bit" and "more verbose"' end up linearly in the main commit history ( https://github.com/scikit-image/scikit-image/commits/main)... which is even worse, isn't it? Am I missing something? Otherwise, I believe in: * adding a line about commit messages in https://scikit-image.org/docs/stable/contribute.html; * spreading the word about good/standard commit messages whenever we get a chance, whether formally or informally (e.g., https://github.com/scikit-image/scikit-image/pull/5479#issuecomment-88194815... ). Cheers, Marianne On Thu, Jul 29, 2021 at 9:28 PM Stefan van der Walt <stefanv@berkeley.edu> wrote:
On Thu, Jul 29, 2021, at 12:08, Alexandre de Siqueira wrote:
some ideas on that: https://youtu.be/ZNaArLZCPMI. I liked the talk, maybe there's something we could use here.
This is good. Here is the written version:
https://blogs.vmware.com/opensource/2021/04/14/improve-your-git-commits-in-t...
Stéfan _______________________________________________ scikit-image mailing list -- scikit-image@python.org To unsubscribe send an email to scikit-image-leave@python.org https://mail.python.org/mailman3/lists/scikit-image.python.org/ Member address: marianne.corvellec@ens-lyon.org