threads / patch / 42640

patchcompletion: add "diff.compactionHeuristic" to git config

Subject: [PATCH] completion: add "diff.compactionHeuristic" to git config

## tl;dr

2 messages between Jun 17, 2016 and Jun 17, 2016. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Martin Vladev· Jun 17, 2016, 16:42 UTC · lore
Signed-off-by: Martin Vladev <martin.vladev@gmail.com>
---
 contrib/completion/git-completion.bash | 1 +
 1 file changed, 1 insertion(+)
Show changes to contrib/completion/git-completion.bash +1 −0
diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
index 3402475..55862d6 100644
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -2066,6 +2066,7 @@ _git_config ()
 		core.whitespace
 		core.worktree
 		diff.autorefreshindex
+		diff.compactionHeuristic
 		diff.external
 		diff.ignoreSubmodules
 		diff.mnemonicprefix
-- 
2.9.0
Junio C Hamano· Jun 17, 2016, 17:04 UTC · re: Martin Vladev · lore

Re: [PATCH] completion: add "diff.compactionHeuristic" to git config

Martin Vladev <martinvladev@gmail.com> writes:
> Signed-off-by: Martin Vladev <martin.vladev@gmail.com>
> ---

The reason why existing documentation marks this feature as experimental is because we would eventually want to end the experiment (and use the better heuristics, if the experiment yields one, unconditionally).

Making it more discoverable by completing like this will make it harder to end the experiment, so I am moderately negative on this change.

Thanks.
Show 15 quoted lines
>  contrib/completion/git-completion.bash | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
> index 3402475..55862d6 100644
> --- a/contrib/completion/git-completion.bash
> +++ b/contrib/completion/git-completion.bash
> @@ -2066,6 +2066,7 @@ _git_config ()
>  		core.whitespace
>  		core.worktree
>  		diff.autorefreshindex
> +		diff.compactionHeuristic
>  		diff.external
>  		diff.ignoreSubmodules
>  		diff.mnemonicprefix

← back to recent threads