Code review comment for lp:~dooferlad/linaro-license-protection/bug1187726

Revision history for this message
Milo Casagrande (milo) wrote :

Looks OK to me.
There is only one typo I guess:

+ """os.path.join with with check that result is inside base_path.

s/with with/with
In case, fix when merging.

review: Approve

« Back to merge proposal