Code review comment for lp:~rvb/maas/bug-1062607

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

I don't see how this can work: "(?P<interface>[\w\-]{0})" will only match a string of length zero. If you want zero or more instances, that's actually easier: "(?P<interface>[\w\-]*)"

I could be missing something, of course. But if not, this suggests a missing test.

review: Needs Fixing

« Back to merge proposal