linaro-license-protection

Code review comment for lp:~pfalcon/linaro-license-protection/crowd-auth

  1. crowd-auth
  2. Merge into trunk
Revision history for this message
Georgy Redkozubov (gesha) wrote :

Looks good.
+1 to Milo's comment on tests, afaics, you've missed a ',' in comparison string on line 202. Should be
202 + "You need to be the member of one of the linaro, batman, catwoman "

Otherwise approved.

review: Approve

« Back to merge proposal