Launchpad itself

Code review comment for lp:~deryck/launchpad/too-much-dupe-email-noise-418659

  1. too-much-dupe-email-noise-418659
  2. Merge into devel
Revision history for this message
Jelmer Vernooij (jelmer) wrote :

Hi Deryck,

One minor thing; it'd be nice if the append argument to saveOldChanges() was explicitly documented in the docstring. Other than that, great to see fewer doctests and more comments and unit tests.

review: Approve (code)

« Back to merge proposal