Launchpad itself

Code review comment for lp:~rockstar/launchpad/recipe-too-new

  1. recipe-too-new
  2. Merge into devel
Revision history for this message
Aaron Bentley (abentley) wrote :

The monkey patch needs to use a try/finally block to ensure the new value doesn't leak, or better yet, use a context manager for the purpose.

Other than that, looks fine.

review: Approve

« Back to merge proposal