Re: [RFC PATCH 0/5] Introduce -t, --table for status/add commands

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

 



On 2023-10-23 19:30, Jacob Stopak wrote:
On Mon, Oct 23, 2023 at 04:34:15PM +0200, Dragan Simic wrote:
On 2023-10-23 12:52, Oswald Buddenhagen wrote:
i for one think that it would be a perfectly valid experiment to go
all-in and beyond with jacob's proposal - _and make it the default_
(when the output is a tty). more advanced users who feel annoyed would
be expected to opt out of it via configuration, as they are for the
advice messages. because it's really the same idea, only thought
bigger.

I'd never support that, FWIW.

FWIW, I'd _never suggest_ that. I very much value Git's current usage
and wouldn't dream to make this the default. This proposal is for an
optional flag to help users who would benefit from it, nothing more,
nothing less. Speculating on user motives to classify them into 2 broad
categories in order to prove the feature isn't helpful misses the point
that there is a (relatively large IMO) subset of users who would benefit
from it.

As an optional flag, experienced users wouldn't bat an eyelash, and the
type of users who installed my tool could use the flag on and off until
they feel confident enough to drop it. But it is always there in case
they need a refresher.

Perhaps my broad classification of git users wasn't that great, and I'm actually happy for being wrong there.

Just to clarify, in general I'd support the inclusion of the table output formatting, but I'd need to test it in detail before saying a definite yes. It would also need to be more feature-complete, and the original author of the patches should commit to maintaining it as the new git feature. Having different options available can't hurt, IMHO.

However, having that as the default is something I'll never support. All this is just my opinion.




[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