launchpad-reviewers team mailing list archive

[Aaron Bentley <aaron@xxxxxxxxxxxxx>]

Review: Approve

This is definitely an improvement.  Please ensure a bug is filed about the lack of a subscribed_by field in SpecificationSubscription, and reference it in the security adapter and test comments.  Please fix the too-long lines.  You can do this by assigning the enum to a local variable.
-- 
https://code.launchpad.net/~adeuring/launchpad/bug-1056881/+merge/135374
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.