Code review comment for lp:~allanlesage/indicator-messages/TDD

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

Allan, I don't usually worry about indentation but the indentation in this patch is actively confusing... it looks like a "fi" is missing but isn't. Could you line up the if..elif..fi statements with each other?

review: Needs Fixing

« Back to merge proposal