Code review comment for lp:~ted/indicator-messages/sanity-testing

Revision history for this message
Charles Kerr (charlesk) wrote :

A few inline comments below, no showstoppers.

Overall I'm very impressed with the IndicatorFixture you've got here.

If I was the one who reviewed it before in i-sound, maybe it was before my time on autopilot and I didn't realize what I was seeing.

When you get back from Galveston, let's look into making a shared repo so that other indicators can use this.

« Back to merge proposal