Code review comment for lp://qastaging/~vivekys/nova/lp790174

Revision history for this message
Todd Willey (xtoddx) wrote :

Extra newline on line 65. I think there should be more indents on 79, 80, 86, 107, 108, and 114. I don't have a stack to try this on, but it looks inline with what the vmware operations look like elsewhere. I've requested a review from the vmware lieutenant for better feedback and hopefully some testing.

review: Needs Fixing

« Back to merge proposal