Change in vdsm[master]: Yet another bunch of completely pep8 fixed files.

asegurap at redhat.com asegurap at redhat.com
Fri Dec 14 13:48:00 UTC 2012


Antoni Segura Puimedon has posted comments on this change.

Change subject: Yet another bunch of completely pep8 fixed files.
......................................................................


Patch Set 1:

Darn! There is an error in pep8 1.3.3 (last tagged relase) that makes full paths not to be checked in the excluded list, only the basename. This was fixed in:
    https://github.com/jcrocholl/pep8/commit/504660cb373315eb6e147390d9e6af78c0d7f4f8

In the meantime, maybe I should just specify the files to be checked by full path.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2d0e38fd43bd76febf3bfb231c4257298e6d921f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list