Change in vdsm[master]: build: Forbid bare except:

ybronhei at redhat.com ybronhei at redhat.com
Tue Dec 17 12:44:40 UTC 2013


Yaniv Bronhaim has posted comments on this change.

Change subject: build: Forbid bare except:
......................................................................


Patch Set 3:

jenkins job with that is redundant, currently we don't check syntax during the make, that's why we have pep8 and pyflaks running on the py files..if pep8 finds it valid, we also do.. document it as a standard and -1 on it is valid as well, but we can't make it mandatory imo

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b7f33b6eb1c90610b3a5d053ba65ce3fc6b74fa
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list