[PATCH 1 of 2] config: rename allowpull to allow-pull

Yuya Nishihara yuya at tcha.org
Sun Nov 12 03:16:06 EST 2017


On Fri, 10 Nov 2017 15:09:21 +0100, David Demelier wrote:
> # HG changeset patch
> # User David Demelier <markand at malikania.fr>
> # Date 1508406199 -7200
> #      Thu Oct 19 11:43:19 2017 +0200
> # Node ID 4157480049dabdf7bd54160c909baf13ea8d5107
> # Parent  407ec7f3ff02ae357074104fe892adc2b12b42a7
> config: rename allowpull to allow-pull
> 
> As part of ConfigConsolidationPlan [1], rename the option according to
> the new UI guidelines [2] and add an alias for backward compatibility.
> 
> [1]: https://www.mercurial-scm.org/wiki/ConfigConsolidationPlan
> [2]: https://www.mercurial-scm.org/wiki/UIGuideline#adding_new_options

Are we agreed to do that for all config options? or is it only for allowpull
and allowpush?

> diff -r 407ec7f3ff02 -r 4157480049da mercurial/help/config.txt
> --- a/mercurial/help/config.txt	Tue Nov 07 13:48:33 2017 -0800
> +++ b/mercurial/help/config.txt	Thu Oct 19 11:43:19 2017 +0200
> @@ -2287,7 +2287,7 @@
>      revisions.
>      (default: False)
>  
> -``allowpull``
> +``allow-pull``
>      Whether to allow pulling from the repository. (default: True)

Nit: Perhaps we should list the deprecated name as well.


More information about the Mercurial-devel mailing list