← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~cprov/launchpad/p3a-api-token into lp:launchpad

 

Review: Approve code

28	+ if len(list(subscription)) == 0:
29	+ return None

This should be "if subscriptions.is_empty():". Would it also perhaps be better to raise Unauthorized?
-- 
https://code.launchpad.net/~cprov/launchpad/p3a-api-token/+merge/210344
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.


References