Code review comment for lp:~jcsackett/launchpad/bugtasks-deacitvated-context-632847

Revision history for this message
Graham Binns (gmb) wrote :

Just one comment: test_listing_seen_with_permission needs a comment at the start of it stating the expected behaviour (it's obvious from the test, but I don't want to have to parse the test to understand what it's testing).

Other than that this is great; r=me.

review: Approve (code)

« Back to merge proposal