Dan Kenigsberg has submitted this change and it was merged. Change subject: PEP8: fix variours errors ......................................................................
PEP8: fix variours errors pep8 1.4.6 gives the following error tests/hooksTests.py:140:23: E127 continuation line over-indented for visual indent tests/hooksTests.py:141:18: E124 closing bracket does not match visual indentation And pep8 1.4.5 on RHEL6 gives the following error vdsm/guestIF.py:30:5: E122 continuation line missing indentation or outdented This patch fixes the errors while makes pep8 1.4.5 and 1.4.6 both agree with the fix. Change-Id: Ib6f8843f7bf9f57c040b4747e89ce90ee6122823 Signed-off-by: Zhou Zheng Sheng <[email protected]> Reviewed-on: http://gerrit.ovirt.org/16424 Reviewed-by: Yaniv Bronhaim <[email protected]> Tested-by: Yaniv Bronhaim <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M Makefile.am M tests/hooksTests.py M vdsm/guestIF.py 3 files changed, 9 insertions(+), 7 deletions(-) Approvals: Yaniv Bronhaim: Verified; Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16424 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib6f8843f7bf9f57c040b4747e89ce90ee6122823 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Kiril Nesenko <[email protected]> Gerrit-Reviewer: Ohad Basan <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
