[PATCH 5 of 5] template: add successors template

Yuya Nishihara yuya at tcha.org
Fri Jul 7 21:14:03 EDT 2017


On Fri, 07 Jul 2017 09:35:48 -0700, Sean Farley wrote:
> Yuya Nishihara <yuya at tcha.org> writes:
> > On Thu, 06 Jul 2017 11:05:22 -0700, Sean Farley wrote:
> >> Augie Fackler <raf at durin42.com> writes:
> >> > On Wed, Jul 05, 2017 at 10:53:58PM +0200, Boris Feld wrote:
> >> >> # HG changeset patch
> >> >> # User Boris Feld <boris.feld at octobus.net>
> >> >> # Date 1499073720 -7200
> >> >> #      Mon Jul 03 11:22:00 2017 +0200
> >> >> # Node ID 82e2b4eb96b573dde890d725d758dbbc49407133
> >> >> # Parent  870bfaafd90e030b85b869922050be98f000a9a9
> >> >> # EXP-Topic successorstemplate
> >> >> template: add successors template
> >> >
> >> > queued, thanks
> >> >
> >> >>
> >> >> Add a 'successorssets' template that returns the list of all closest known
> >> >> sucessorssets for a changectx. The elements of the list are changesets.
> >> >
> >> > I'm not...super in love with the name successorssets, but I have no
> >> > better ideas. I'd welcome some bikeshedding in the next ten days.
> >> 
> >> Off the top of my head:
> >> 
> >> successorgroup
> >> successorlike
> >> similarsuccessors
> >> successorkin
> >
> > No idea which is better, but a name like "similarsuccessors" might be nice
> > to justify calling the inner list as "successors".
> >
> >   {similarsuccessors % "{successors % ...}"}
> >                          ^^^^^^^^^^
> >               sounds okay, and its actually a partial list of successors
> >
> >   {successorgroup % "{group % ...}"}
> >                       ^^^^^
> >               calling it "successors" seems a bit odd because "group" is noun
> 
> I like similarsuccessors, fwiw. Also, "successors" is a noun as well.

I meant "successor" in "successorgroup" is adjective, so can't be a name
pointing to each element.


More information about the Mercurial-devel mailing list