Zhou Zheng Sheng has posted comments on this change.

Change subject: Make utils pep8 compliant
......................................................................


Patch Set 1: Verified; Looks good to me, but someone else must approve

Pass autobuild.sh. pydiff finds some changes, the first one splitting import 
into separate ones in line 31 vs 31, the second one is introducing a temp 
variable in line 492 vs 519, the last one is changing "== "to "is" when 
comparing identity to "None" in line 656 vs 689. All changes are acceptable.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I514660d73071f8fb417fea0cc54973a55002992f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to