Launchpad itself

Code review comment for lp:~jtv/launchpad/bug-848954

  1. bug-848954
  2. Merge into devel
Revision history for this message
Graham Binns (gmb) wrote :

[1]

71 + raise AssertionError(
72 + "%s killed us on %s %s" % (len(q), query, args))

I know you didn't write it, but this error message makes little sense to me... any mileage in changing it in this branch, or does it make sufficient sense to those In The Know that it should remain untouched?

review: Approve (code)

« Back to merge proposal