[Merge] lp:~muktupavels/compiz/ccsm-key-error into lp:compiz

Dmitry Shachnev mitya57 at gmail.com
Tue Jul 17 10:07:52 UTC 2018


Review: Approve

Tip for next time: if you add parentheses then you don't need to escape newlines:

if (self.CurrentPlugin is not None
    and self.CurrentPlugin.Name in self.Results
    and self.CurrentGroup in self.Results[self.CurrentPlugin.Name]):
    ...

Like you wrote is also fine though.
-- 
https://code.launchpad.net/~muktupavels/compiz/ccsm-key-error/+merge/349666
Your team Compiz Maintainers is subscribed to branch lp:compiz.



More information about the Ubuntu-reviews mailing list