Code review comment for lp:~lifeless/launchpad/bug-421901

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

As discussed on IRC, for /me/ superuser_connection() is a clearer name than root_connection() ('root' is a touch overloaded and I didn't get the intent in this context).

Other than that, looks fine.

review: Approve

« Back to merge proposal