Code review comment for lp://staging/~angeloc/unity-lens-files/fix-for-773841

Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

Btw Angelo, have you signed the contributor agreement? We need that in order to be able to take your branch http://www.canonical.com/contributors

On merging of the url checker classes I think Michal meant to have two different instances of the same class, but being able to define which regexes they use in the constructor or something. Michal?

« Back to merge proposal