Code review comment for lp:~jml/launchpad/expose-get-owners

Revision history for this message
Jonathan Lange (jml) wrote :

From the bug report:

jml says "Note that exposing this method, which should be a very, very simple patch, is actually quite tedious. The method in question returns IPillarName, which is itself not exposed via the API yet"

leonardr replies "You don't need to publish internal Launchpad methods exactly as is. You should be able to define a new method that returns pillar objects (which are already published), and publish that as a named operation."

Also there are conflicts.

review: Needs Fixing

« Back to merge proposal