D1293: rebase: use fm.formatlist() and fm.formatdict() to support user template

yuja (Yuya Nishihara) phabricator at mercurial-scm.org
Sat Nov 4 03:50:10 EDT 2017


yuja requested changes to this revision.
yuja added a comment.
This revision now requires changes to proceed.


  Can you add a test that should be fixed by this patch?
  Perhaps that will show we'll probably want to set the name of dict item pair
  other than the default 'key'/'value'.

INLINE COMMENTS

> rebase.py:1550
> +        fd = fm.formatdict
> +        nodechanges = fd({hf(oldn): fl([hf(n) for n in newn], 'succs')
> +                          for oldn, newn in replacements.iteritems()})

The name argument is for each item, not for the list. Perhaps,
it should be name='node' (or name ='succ').

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D1293

To: pulkit, #hg-reviewers, yuja
Cc: yuja, mercurial-devel


More information about the Mercurial-devel mailing list