On Mon, 26 Feb 2018 22:05:27 +0800, Anton Shestakov wrote:
> # HG changeset patch
> # User Anton Shestakov <a...@dwimlabs.net>
> # Date 1519649061 -28800
> #      Mon Feb 26 20:44:21 2018 +0800
> # Node ID 026f4c5a52a0070c5b712cad247818e6566d4748
> # Parent  0aa1728931cc2c2c7d6ee0f18e0618fc17add42a
> hgweb: explain instabilities of unstable changesets
> 
> Let's bikeshed a bit more before I touch other hgweb styles or start adding
> non-hgweb template keywords/functions:
> 
> - I like "explainunstable", but it works as a function name, and not so much 
> as
>   a keyword name. Other options so far are: whyunstable, reasonsunstable,
>   unstablereasons, unstablesources.

"whyunstable" or "unstablereasons" seems good to me. The latter can imply it
returns a list.

> - What would be the best way to implement this feature in regular templates?
>   Since there can be multiple instabilities, it needs to be a list, and then
>   content-divergent changesets also have a list of divergent nodes... Is there
>   a better way to do this than what's done in this patch?

Perhaps that depends on how we want to write a template. If we want to list
reasons per instability for example, it's probably better to return a dict
of lists.

  unstablereasons.orphan % '{node} {reason} ...'
  unstablereasons.contentdivergent % '{divergentnodes % "{node}"} {reason} ...'
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to