Review: Approve


Diff comments:

> 
> === modified file 'lib/lp/scripts/garbo.py'
> --- lib/lp/scripts/garbo.py   2015-07-13 00:52:36 +0000
> +++ lib/lp/scripts/garbo.py   2015-07-21 07:05:40 +0000
> @@ -1689,6 +1713,7 @@
>          CodeImportEventPruner,
>          CodeImportResultPruner,
>          GitJobPruner,
> +        WebhookJobPruner,

This list is almost alphabetised but not quite.  Maybe it could be entirely 
alphabetised.

>          HWSubmissionEmailLinker,
>          LiveFSFilePruner,
>          LoginTokenPruner,


-- 
https://code.launchpad.net/~wgrant/launchpad/webhook-deletion/+merge/265360
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.

_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help   : https://help.launchpad.net/ListHelp

Reply via email to