yuja requested changes to this revision.
yuja added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> releasenotes.py:246
> +def _getadmonitionlist(ui, sections):
> +    for section in sections._sections:
> +        ui.write(_(str(section[0]) + ": " + section[1] + "\n"))

Perhaps this can be just `for section in sections` since
`releasenotessections` implements `__iter__`.

> releasenotes.py:247
> +    for section in sections._sections:
> +        ui.write(_(str(section[0]) + ": " + section[1] + "\n"))
> +

Delete `_()`, which is the marker for strings to be translated.

`"%s: %s\n" % ...` should be better for python 3 compatibility.

> releasenotes.py:523
> +
> +    rev = opts.get('rev')
>  

merge error?

REPOSITORY
  rHG Mercurial

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

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

Reply via email to