From: Kristoffer Haugsbakk <kristofferhaugsbakk@fastmail.com>
Date: Wed, 21 May 2025 14:49:10 GMT
Subject: Re: [PATCH 6/6] doc: notes: treat --stdin equally between copy/remove
Message-ID: <8ddf10d7-d666-4298-8319-7feba1af85a1@app.fastmail.com>
In-Reply-To: <xmqq34czhyz8.fsf@gitster.g>

On Tue, May 20, 2025, at 21:19, Junio C Hamano wrote:
>> +	For `remove` and `copy`. See the respective subcommands. This
>> +	option can be combined with object names given via the command
>> +	line for `remove`. However, this is not the case for `copy`.
>
> During my first read of this paragraph, I somehow read the first two
> sentences as "if you want to learn how this option behaves with the
> remove and copy subcommands, go read the documentation for these
> subcommands", making me anticipate that the remainder of the
> paragraph would talks about what happens when --stdin is given to
> other subcommands.  Apparently that is not what is going on.
>
> It may be simpler to just say
>
> 	Only valid for `remove` and `copy`.  See the respective
> 	subcommands.
>
> and stop there.

Yep I agree.  Just leave it to the subcommand sections.

-- 
Kristoffer Haugsbakk