From: Randall S. Becker <rsbecker@nexbridge.com>
Date: Mon, 05 Jul 2021 14:02:57 GMT
Subject: RE: [PATCH 5/5] config: add default aliases
Message-ID: <044e01d771a6$7c8cefc0$75a6cf40$@nexbridge.com>
In-Reply-To: <5cbb845f-b8a6-939e-cf37-a3b375438616@mfriebe.de>

On July 2, 2021 6:38 PM, martin wrote:
>To: Felipe Contreras <felipe.contreras@gmail.com>; Ævar Arnfjörð Bjarmason <avarab@gmail.com>
>Cc: git@vger.kernel.org; Junio C Hamano <gitster@pobox.com>
>Subject: Re: [PATCH 5/5] config: add default aliases
>
>On 02/07/2021 23:58, Felipe Contreras wrote:
>> Ævar Arnfjörð Bjarmason wrote:
>>>> +ALIAS
>>>> +~~~~~
>>>> +'git rb'
>>> So 'r'e'b'ase, not 're'base.
>> I don't know if 're' makes more sense here.
>
>re:
>restore
>rebase
>reset
>
>And restore is on the level of checkout => so more important.

I do not want anything helping out the use of rebase, which we actively discourage in our shop - except for rebase --autosquash to fix up topic branches for delivery. git 're' is certainly not helpful.

From an earlier suggestion, why not just put all of your desired aliases in its own file somewhere and reference them through a construct in .gitconfig like:

include="/path/to/alias-config"

which would have to be implemented, but that decouples alias definitions from core git code and allows sharing of the definitions by a team without impinging on anyone else. I have great trepidation that users are going to start writing scripts using these aliases. I am going to be implementing a team standards document that would cause any use of aliases in scripts to fail code reviews - in fact, I'm looking to implement a commit hook that rejects the use of aliases in scripts that are committed.