[PATCH 1 of 3 STABLE V2] pager: rename 'pager.enable' to 'ui.pager'

Yuya Nishihara yuya at tcha.org
Mon May 1 22:34:50 EDT 2017


On Mon, 1 May 2017 21:22:02 -0400, Augie Fackler wrote:
> On Mon, May 01, 2017 at 10:29:21PM +0200, Pierre-Yves David wrote:
> > # HG changeset patch
> > # User Pierre-Yves David <pierre-yves.david at ens-lyon.org>
> > # Date 1493649410 -7200
> > #      Mon May 01 16:36:50 2017 +0200
> > # Branch stable
> > # Node ID 853bf9ca7b3b1f4d8f54ed8c60ff92371132c097
> > # Parent  ad70a36b5b472c295bfdb849ba69eac871305b36
> > # EXP-Topic pager
> > # Available At https://www.mercurial-scm.org/repo/users/marmoute/mercurial/
> > #              hg pull https://www.mercurial-scm.org/repo/users/marmoute/mercurial/ -r 853bf9ca7b3b
> > pager: rename 'pager.enable' to 'ui.pager'
> 
> Mentioned this on the other thread, but in brief here so it doesn't
> get overlooked: I think ui.pager is confusing as a boolean when
> ui.editor already exists and is a program to invoke. They sound
> related to me.

Agreed. pager.enable is a bit unfortunate, but 'ui.pager = <bool>' is more
confusing.


More information about the Mercurial-devel mailing list