Code review comment for lp:~allenap/maas/urlencode-is-duckist

Revision history for this message
Julian Edwards (julian-edwards) wrote :

Looks ok.

34 + # The tuple is important because this is used as input to urlencode,
35 + # which refuses to let ducks quack.

I'd expand this to include this explanation you've got in the commit msg about urlencode's requirement for a tuple. It just reads as a rant otherwise :)

review: Approve

« Back to merge proposal