Launchpad itself

Code review comment for lp:~intellectronica/launchpad/search-for-bugs-affecting-me

  1. search-for-bugs-affecting-me
  2. Merge into devel
Revision history for this message
Deryck Hodge (deryck) wrote :

Looks good to me. As mentioned at the sprint, I think we need something to clarify what you're doing with the reduce statement, just so its easy to scan the test and get what's going on. It wasn't obvious to me on first glance.

Otherwise, well done. Thanks!

Cheers,
deryck

review: Approve (code)

« Back to merge proposal