Re: [PATCH 5/5] config: add default aliases

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

 



On 02/07/2021 23:02, Felipe Contreras wrote:

If I was a committer on this project, I would have to be much more
convinced that there is long-term value in this series than appears on
the surface.
  1. It doesn't affect anyone negatively
  2. You don't have to use them if you don't want to
  3. They don't affect your aliases, even if they have the same name
  4. Everyone has aliases
  5. Every SCM in history has had aliases

What more would you need?


Well, it might be good if they were configurable.

core.built-in-alias=false
core.built-in-alias=true
core.built-in-alias=sw,rs  # switch,restore

Also, it can be debated if they should be on or off by default. (hence built-in)

If the setting is undefined, then if a user does
   git sw branch
It will print:
built in aliases are not enabled. Please run
  git config core.built-in-alias=true

That way no one is forced to anything, but they are easy to enable, and self advertising.




[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