From: Junio C Hamano Date: Wed, 30 Jul 2014 18:52:03 GMT Subject: Re: [PATCH 3/5] checkout --to: no auto-detach if the ref is already checked out Message-ID: In-Reply-To: Junio C Hamano writes: > Michael J Gruber writes: > >> As an error message that is completely sufficient. >> >> The advice messages are meant to teach the user about the normal parts >> of the toolchest to use in a situation of "conflict", aren't they? > > Not really. They are to remind (to those who learned but forgot) > and to hint (to those who haven't realized they have things to learn > in this area). Wall of text that tries to do more than that, like > "teaching", risks not getting read by anybody. > >> Maybe >> we should ask someone who hasn't turned them off... > > Actually, I run without most of the 'advice.*' turned off. Ehh, what I meant was that I do not have "advice.foo = false" for most of 'foo', i.e. I run with most of them still active.