launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #12585
Re: [Merge] lp:~abentley/launchpad/hide-sprint-blueprints into lp:launchpad
Review: Approve
This looks alright.
As one minor point, it might be cleaner to set the default argument for user in `specifications` to None, rather than passing in None everytime you need it. But I don't see that as a blocker to approval--it's just a thought.
--
https://code.launchpad.net/~abentley/launchpad/hide-sprint-blueprints/+merge/126792
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.
Follow ups
References