Re: [PATCH v4 0/3] advice: add "all" option to disable all hints

From: Dragan Simic <dsimic@manjaro.org>
Date: May 3, 2024, 07:31 UTC
Message-ID: <ab604f4b73eab1d9aae4e2a20ba0630e@manjaro.org>
In-Reply-To: <20240503071706.78109-1-james@jamesliu.io>

Hello James,

On 2024-05-03 09:17, James Liu wrote:

Show 14 quoted lines

>  Documentation/git.txt | 18 +++++++-----
>  advice.c              |  7 +++++
>  environment.h         |  7 +++++
>  git.c                 | 11 +++++--
>  t/t0018-advice.sh     | 68 +++++++++++++++++++++++++++++++++++++++++++
>  5 files changed, 101 insertions(+), 10 deletions(-)
> 
> Range-diff against v3:
> 1:  55d5559586 < -:  ---------- advice: add --no-advice global option
> -:  ---------- > 1:  ae3f45dadc doc: clean up usage documentation for
> --no-* opts
> -:  ---------- > 2:  1b0019026a doc: add spacing around paginate 
> options
> -:  ---------- > 3:  31e73e6c0e advice: add --no-advice global option

Just a small suggestion... Perhaps the creation factor needs adjusting for the range diff to actually be produced. To be clear, I don't mind that it's missing here.