Re: [RFC PATCH v2 0/6] Noobify format for status, add, restore

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

 



On 2023-10-28 19:35, Jacob Stopak wrote:
On Sat, Oct 28, 2023 at 06:20:41PM +0200, Dragan Simic wrote:
I agree, there should be a well-placed hint, but we'd need to think really well where to place it, so we don't annoy experienced git users too much,
while we also inform the less experienced users.

Yes, hmm, I wonder if maybe we could add the hint for the extended option
only in the case that the user uses the --verbose option either on the
command line or via the config setting. Since using the verbose option
tells us the user is asking for more details, that might be a good time
to inform them of the _even more detailed_ option, but of course that
hint could be disabled easily if they preferred the "basic" verbosity.

That's something we can keep thinking about, until we find a good solution. Also, a detailed review of the current logic behind displaying the hints should be performed first, if you agree.

Sounds good, thank you. If you agree, I'll go ahead and implement support for a few "<command>.verbose" configuration options during the next week or so, and submit the patches. I'll most probably come to some more important
conclusions while implementing that, which I'll relay over, of course.

Yes I agree, that sounds great! Maybe I'll just wait then until seeing
your implementation of that before I poke around on mine more. Then I'll
apply your patches locally to add my extended option.

Great, thanks. I'll start working on the patches tomorrow or so, and I'll get back with any important conclusions or open questions arising from that, so we can discuss them further.




[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