Re: [PATCH v1 00/11] And new command "restore"

From: Jacob Keller <jacob.keller@gmail.com>
Date: Mar 10, 2019, 22:45 UTC
Message-ID: <6B5DF0A1-DCEB-45C2-B205-E3CF7469B7C6@gmail.com>
In-Reply-To: <CACsJy8BB1GAaBX=YcmNf1fLj5chc5hcvHLTwvzwJgvSWWKNFdQ@mail.gmail.com>

On March 10, 2019 4:19:28 AM PDT, Duy Nguyen <pclouds@gmail.com> wrote:

Show 13 quoted lines

>On Fri, Mar 8, 2019 at 5:17 PM Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
>wrote:
>> - --index has a different meaning in git-apply. I don't have a good
>>   suggestion except "yeah we have two different worlds now, the old
>>   and the new one"
>
>I will rename --index to --staged to avoid this. It is already used as
>synonym for --cached in git-diff. I will also add --staged as synonym
>to "git rm" so that "git status" can consistently advise "git <verb>
>--staged" where verb is either restore or rm.
>
>Since option rename is a lot of work. If you think this is not a good
>move, raise it now, even if it's just "I don't like it, no reasons".

For what it's worth, I think this is a good rename.

Thanks, Jake

-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.