← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~cjwatson/launchpad/dead-code into lp:launchpad

 

Review: Approve code



Diff comments:

> 
> === modified file 'lib/lp/bugs/browser/bugtracker.py'
> --- lib/lp/bugs/browser/bugtracker.py	2015-07-08 16:05:11 +0000
> +++ lib/lp/bugs/browser/bugtracker.py	2015-09-28 21:49:16 +0000
> @@ -550,16 +549,6 @@
>          return u"Bug trackers"
>  
>  
> -class BugTrackerBreadcrumb(Breadcrumb):
> -    """Builds a breadcrumb for an `IBugTracker`."""
> -
> -    rootsite = None
> -
> -    @property
> -    def text(self):
> -        return self.context.title

This could perhaps be registered instead, though it isn't critical.

> -
> -
>  @implementer(IRemoteBug)
>  class RemoteBug:
>      """Represents a bug in a remote bug tracker."""


-- 
https://code.launchpad.net/~cjwatson/launchpad/dead-code/+merge/272670
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.


References