Can we define the type (for this and any others) in an "Instance variables" on the class body?

I made a similar comment in another change, but I'd like to keep the original form, for the same reason that we don't put } or ] on the final line for dicts/lists. It makes it easier to modify/add lines on multi-line strings without affecting previous lines.
Also has a nice symmetry.
Doesn't necessarily matter in this particular case, but I'd prefer it as a general guideline.

This should be in the multi-line form.

We're getting a generator at the end, not a tuple. Probably isn't any better than a list (maybe worse).
Though, we're making an assumption here that we probably should check/handle with an exception handler. If we get too few or too many items here, we're going to crash. It's not a new problem, but worth considering.

please, do not remove that parameter. I added this in /r/11102

Nit: These are all missing trailing periods.

These comments are missing trailing periods.

Too many blank lines in the comment.

Double spaces should be a single space after the period.

The single backticks must be double backticks for ReST.
Also the period should be after the parens.

We should transition to keyword-only arguments here, given the mix of kwargs for the parent and an argument here.

"Mercurial"

The type is already defined on the class body.

We have a more explicit type we can list here now.

Same here.

Ship It!