Code review comment for lp:~bac/launchpad/bug-5927

Revision history for this message
Gavin Panella (allenap) wrote :

Cool :) +1

A couple more really really minor things.

[12]

+ def _get_bug_tasks(self):
+ store = Store.of(self.bug)
+ return store.find(
+ BugTask, BugTask.bug == self.bug)

Perhaps a docstring to make it obvious what this is for?

[13]

+ bugtasks = [bugtask.assignee.name for bugtask in bugtasks]

Strictly this is a list of assignee names.

This appears twice.

review: Approve

« Back to merge proposal