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.
« Back to merge proposal
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.