Change in vdsm[master]: Fixing pep8 in vdsm/hooks.py

zhshzhou at linux.vnet.ibm.com zhshzhou at linux.vnet.ibm.com
Mon May 21 03:30:53 UTC 2012


Zhou Zheng Sheng has posted comments on this change.

Change subject: Fixing pep8 in vdsm/hooks.py
......................................................................


Patch Set 1: (1 inline comment)

Please refer to the inline comments in line 39 of hooks.py.

....................................................
File vdsm/hooks.py
Line 39: 
I think there is no need to insert blank lines between a function and its comments. Blank lines can be inserted above the function. What do you think?

--
To view, visit http://gerrit.ovirt.org/4537
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I79717299c1b75df3ff1f429648f8d1dd225f0d70
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Bing Bu Cao <mars at linux.vnet.ibm.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: ShaoHe Feng <shaohef at linux.vnet.ibm.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list