yuja added a comment.

  > +def filterknowncreateopts(ui, createopts):
  >  +    """Filters a dict of repo creation options against options that are 
known.
  
  Nit: I thought this would select only known options.
  
  > +        raise error.Abort(_('unable to create repository because of 
unknown '
  >  +                            'creation option: %s') %
  >  +                          ', '.sorted(unknownopts.keys()),
  
  Removed `.keys()` to make it clearer that we don't need a new list on Py2.

REPOSITORY
  rHG Mercurial

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

To: indygreg, #hg-reviewers
Cc: yuja, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to