Code review comment for ~lihow731/plainbox-provider-checkbox/+git/plainbox-provider-checkbox:master

Revision history for this message
Pierre Equoy (pieq) wrote :

+1, thanks!

For the

self.logger.error("Unable to control recording volume. "
                               "Test results may be wrong")

line, I meant adding a period at the end of the sentence:

self.logger.error("Unable to control recording volume. "
                               "Test results may be wrong.")

But it's a nitpick. (and thanks for fixing the spae by the way :))

review: Approve

« Back to merge proposal