launchpad-reviewers team mailing list archive

[Benji York <benji@xxxxxxxxxxxxx>]

I may be misunderstanding, but this bit (which exists in many different
places, but this is representative) doesn't seem right to me:

    :param sponsored: The user who is being sponsored to make the copy.

>From the MP and the behavior, it seems to me that it is not a person
being sponsored, but a person who is sponsoring (sponsoring the copy).
If so, the attribute name(s) of "sponsored" would better be "sponsor"
and the description something like "The user who is sponsoring this
copy."
-- 
https://code.launchpad.net/~julian-edwards/launchpad/sponsor-syncs-bug-827555/+merge/84287
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~julian-edwards/launchpad/sponsor-syncs-bug-827555 into lp:launchpad.