← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~cprov/launchpad/bug-1295318 into lp:launchpad

 

Review: Approve code

56	+ """API wrapper for getPublishedSources.
57	+
58	+ It loads additional related objects only needed in the API call
59	+ context (i.e. security checks and entries marshalling).
60	+ """

This will appear as the API documentation for getPublishedSources. You probably want to mention the APIness in a comment, not the docstring.


-- 
https://code.launchpad.net/~cprov/launchpad/bug-1295318/+merge/216731
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.


References