Code review comment for lp:~abentley/launchpad/hide-sprint-blueprints

Revision history for this message
j.c.sackett (jcsackett) wrote :

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.

review: Approve

« Back to merge proposal