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

INLINE COMMENTS

> bookmarks.py:859
> +            if ctx.hidden():
> +                repo.ui.warn(_("accessing hidden changeset %s\n") % ctx)
> +            tgt = ctx.node()

Perhaps we can say "bookmarking" or "reviving" instead of "accessing"? I don't 
know which is better, though.

Another minor problem I found is this warning will be repeated if
you pass more than one bookmark names.

REPOSITORY
  rHG Mercurial

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

To: pulkit, #hg-reviewers, yuja
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