Re: [PATCH] checkout: do not mention detach advice for explicit --detach option

2016-08-11 Thread Junio C Hamano
Stefan Beller writes: > On Wed, Aug 10, 2016 at 10:32 AM, Jonathan Nieder wrote: >> Stefan Beller wrote: >> >>> When a user asked for a detached HEAD specifically with `--detach`, >>> we do not need to give advice on what a detached HEAD state entails as

Re: [PATCH] checkout: do not mention detach advice for explicit --detach option

2016-08-10 Thread Stefan Beller
On Wed, Aug 10, 2016 at 10:32 AM, Jonathan Nieder wrote: > Stefan Beller wrote: > >> When a user asked for a detached HEAD specifically with `--detach`, >> we do not need to give advice on what a detached HEAD state entails as >> we can assume they know what they're getting

[PATCH] checkout: do not mention detach advice for explicit --detach option

2016-08-10 Thread Stefan Beller
When a user asked for a detached HEAD specifically with `--detach`, we do not need to give advice on what a detached HEAD state entails as we can assume they know what they're getting into as they asked for it. Signed-off-by: Stefan Beller --- builtin/checkout.c | 2 +- 1

Re: [PATCH] checkout: do not mention detach advice for explicit --detach option

2016-08-10 Thread Jonathan Nieder
Stefan Beller wrote: > When a user asked for a detached HEAD specifically with `--detach`, > we do not need to give advice on what a detached HEAD state entails as > we can assume they know what they're getting into as they asked for it. Example? Tests? > Signed-off-by: Stefan Beller

Re: [PATCH] checkout: do not mention detach advice for explicit --detach option

2016-08-10 Thread Junio C Hamano
Stefan Beller writes: > When a user asked for a detached HEAD specifically with `--detach`, > we do not need to give advice on what a detached HEAD state entails as > we can assume they know what they're getting into as they asked for it. Makes sense; I agree that "Don't be