Launchpad itself

Code review comment for lp:~lifeless/launchpad/oops

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

This all looks fine. I like the new API! And I'm sure the information will be really useful.

I worry a bit about tests that don't have a participation when they try to access the database, but well you'll find out about that :-) I think the scripts should all be fine (execute_zcml_for_scripts() calls login(ANONYMOUS) iirc).

review: Approve

« Back to merge proposal