launchpad-reviewers team mailing list archive

[Robert Collins <robertc@xxxxxxxxxxxxxxxxx>]

> +    def addManyTasks(self, owner, targets):
> +        """See `IBug`."""
> +        new_tasks = getUtility(IBugTaskSet).createManyTasks(self, owner, targets)
> +        return new_tasks

That method doesn't seem to have any reason to exist... its
indirection for no benefit. I suggest either using it from IBugTaskSet
or not putting it on IBugTaskSet at all and having the body in IBug.

-Rob

-- 
https://code.launchpad.net/~rharding/launchpad/bugnom_874250/+merge/105317
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.