indygreg added inline comments.

INLINE COMMENTS

> wireprotov2server.py:602-605
> +        if 'default' in meta and meta.get('required'):
> +            raise error.ProgrammingError('%s argument for command %s is 
> marked '
> +                                         'as required but has a default 
> value' %
> +                                         (arg, name))

Does this added check not address your review feedback, @durin42?

REPOSITORY
  rHG Mercurial

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

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

Reply via email to