Gary, thanks for the improvements.
This looks much, much, nicer — sorry for putting you through the trouble, but I think it was worth it. :)
review approve merge approve
Just one minor tidbit below:
>=== modified file 'lib/lp/bugs/scripts/tests/test_bugnotification.py' ... >+ >+class MockBugActivity: >+ """A mock BugActivity user for testing."""
A typo most likely: s/user/used/.
Cheers, Danilo
« Back to merge proposal
Gary, thanks for the improvements.
This looks much, much, nicer — sorry for putting you through the
trouble, but I think it was worth it. :)
review approve
merge approve
Just one minor tidbit below:
>=== modified file 'lib/lp/ bugs/scripts/ tests/test_ bugnotification .py'
...
>+
>+class MockBugActivity:
>+ """A mock BugActivity user for testing."""
A typo most likely: s/user/used/.
Cheers,
Danilo