Code review comment for lp://staging/~psivaa/britney/announce-generic-testclient

Revision history for this message
Celso Providelo (cprov) wrote :

Psivaa,

When you arrive this morning, please take a look at https://code.launchpad.net/~cprov/britney/testclient-api/+merge/259668 and check if my refactoring of the announcer (based on the work I've done for the collector) makes sense.

My MP still pending integration tests and better error handling (logging, at least).

« Back to merge proposal