[Geany-devel] Commit messages on merges
Colomban Wendling
lists.ban at xxxxx
Sun Mar 4 15:07:26 UTC 2012
Le 04/03/2012 09:28, Frank Lanitz a écrit :
> On Sun, 04 Mar 2012 03:40:29 +0100
> Colomban Wendling <lists.ban at herbesfolles.org> wrote:
>
>> IMO we should not record merges when there is only one single commit
>> or when the commits are unrelated (though the latter should probably
>> be less common) and rather rebase or cherry-pick the commits.
>>
>> However, we must keep the merge when the commits are a whole thing not
>> to lose that information (when several commits are needed to
>> implement a single thing).
>
> I agree. And in second case we have to keep care that merge message is
> informative enough to don't go into complete tree just to understand
> what have been done there. Personally I started using the git merge
> command from command line more often instead of github's web interface
> as its not satisfying my understanding.
Same for me, moreover because I prefer to test the PR locally as a
simple branch before doing the merge, so it's not much effort than using
the GitHub UI, and it's a lot more powerful.
Cheers,
Colomban
More information about the Devel
mailing list