git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 1/2] advice: Document that they all default to true

From
Junio C Hamano <gitster@pobox.com>
Date
Dec 19, 2011, 05:35 UTC
Message-ID
<7vhb0xnmyi.fsf_-_@alter.siamese.dyndns.org>
In-Reply-To
<7vliq9nn0l.fsf@alter.siamese.dyndns.org>

By definition, the default value of "advice.*" variables must be true and they all control various additional help messages that are designed to aid new users. Setting one to false is to tell Git that the user understands the nature of the error and does not need the additional verbose help message.

Also fix the asciidoc markup for linkgit:git-checkout[1] in the description of the detachedHead advice by removing an excess colon.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
 Documentation/config.txt |   19 ++++++++-----------
 1 files changed, 8 insertions(+), 11 deletions(-)
diff --git a/Documentation/config.txt b/Documentation/config.txt
index 8a7d2d4..9be7106 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -115,35 +115,32 @@ in the appropriate manual page. You will find a description of non-core
 porcelain configuration variables in the respective porcelain documentation.
 
 advice.*::
-	When set to 'true', display the given optional help message.
-	When set to 'false', do not display. The configuration variables
-	are:
+	These variables control various optional help messages designed to
+	aid new users. All 'advice.*' variables default to 'true', and you
+	can tell Git that you do not need help by setting these to 'false':
 +
 --
 	pushNonFastForward::
 		Advice shown when linkgit:git-push[1] refuses
-		non-fast-forward refs. Default: true.
+		non-fast-forward refs.
 	statusHints::
 		Directions on how to stage/unstage/add shown in the
 		output of linkgit:git-status[1] and the template shown
-		when writing commit messages. Default: true.
+		when writing commit messages.
 	commitBeforeMerge::
 		Advice shown when linkgit:git-merge[1] refuses to
 		merge to avoid overwriting local changes.
-		Default: true.
 	resolveConflict::
 		Advices shown by various commands when conflicts
 		prevent the operation from being performed.
-		Default: true.
 	implicitIdentity::
 		Advice on how to set your identity configuration when
 		your information is guessed from the system username and
-		domain name. Default: true.
-
+		domain name.
 	detachedHead::
-		Advice shown when you used linkgit::git-checkout[1] to
+		Advice shown when you used linkgit:git-checkout[1] to
 		move to the detach HEAD state, to instruct how to create
-		a local branch after the fact.  Default: true.
+		a local branch after the fact.
 --
 
 core.fileMode::
-- 
1.7.8.370.gb3269
Previous: Junio C HamanoNext: Jeff King
Message 7 of 11 in “push: --ignore-stale option”
  1. 2/2 push: --ignore-stale optionJunio C Hamano, Dec 13, 2011
  2. Andreas EricssonDec 13, 2011
  3. Jeff KingDec 15, 2011
  4. Marc BranchaudDec 15, 2011
  5. Junio C HamanoDec 15, 2011
  6. Junio C HamanoDec 19, 2011
  7. 1/2 advice: Document that they all default to trueJunio C Hamano, Dec 19, 2011
  8. Jeff KingDec 19, 2011
  9. 2/2 push: hint to use push.default=upstream when appropriateJunio C Hamano, Dec 19, 2011
  10. Jeff KingDec 19, 2011
  11. Junio C HamanoDec 19, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.