Re: [PATCH] sequencer: allow disabling conflict advice

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Philippe

On 09/03/2024 17:53, Philippe Blain wrote:
Le 2024-03-04 à 12:56, Junio C Hamano a écrit :
Phillip Wood <phillip.wood123@xxxxxxxxx> writes:

... So we probably do need a new config variable but
I think it should have a generic name - not be sequencer specific so
we can extend its scope in the future to "git merge", "git am -3",
"git stash" etc.

A very good point.  Thanks for your careful thinking.

OK, I agree we can make the new advice more generic, but I'm lacking
inspiration for the name. Maybe 'advice.mergeConflicted' ?
Or 'advice.resolveConflictedMerge' ? though this is close to the existing
'resolveConflict'...
Maybe just 'advice.mergeConflict' ?

'advice.mergeConflict' sounds good to me

Best Wishes

Phillip





[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux