On Sun, Apr 6, 2025 at 2:52 PM Daniel Gustafsson <dan...@yesql.se> wrote:

>
> Looking at the other variables they tend to use "if set, then" so we should
> probably stick to that for this as well?  Something like the below perhaps?
>
>         HELP0("  WATCH_INTERVAL\n"
> -                 "    number of seconds \\watch by default waits between
> executing the query buffer\n");
> +                 "    if set to an interval value, overrides the default
> two second \\watch interval\n");
>
>
Using "interval" here could cause minor confusion since it is a valid core
data type but not accepted here.

psql --set=WATCH_INTERVAL='10m' postgres
psql: error: invalid value "10m" for "WATCH_INTERVAL"

select '10m'::interval;
 interval
----------
 00:10:00

How about:

+ "    if set to a number, overrides the default two second \\watch
interval\n"

I do like the consistency of "if set" even though most of the others are
booleans.

David J.

Reply via email to