Code review comment for lp:~javier.collado/utah/url-doctest

Revision history for this message
Max Brustkern (nuclearbob) wrote :

I don't see an ellipsis that would match the +ELLIPSIS directive on line 209 of the merge, so I wonder if that directive is needed.

Other than that, it looks reasonable to me.

review: Needs Information

« Back to merge proposal