Launchpad itself

Code review comment for lp:~stub/launchpad/librarian-gc

  1. librarian-gc
  2. Merge into devel
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

As said on IRC, a comment about the timescale of "recent" in this context in the tests would be great.

It would also be nice to not have SQL in string constants and key off a config value not a literal '1 week' -- but probably not in this branch.

review: Approve

« Back to merge proposal