Change in vdsm[master]: lint: providing a default pylintrc
danken at redhat.com
danken at redhat.com
Sat Aug 10 05:30:07 UTC 2013
Dan Kenigsberg has submitted this change and it was merged.
Change subject: lint: providing a default pylintrc
......................................................................
lint: providing a default pylintrc
We can use pylint to enable some further checks on the codebase.
This can help up to have a cleaner codebase. The proposed pylintrc
is adapted from the one used by the Python language to let us
use java-like coding conventions for functions, methods and such.
Change-Id: Ia41d2bf225e0b5e47a0e2718b906fbd44f5a54e9
Signed-off-by: Giuseppe Vallarelli <gvallare at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/17772
Reviewed-by: Sandro Bonazzola <sbonazzo at redhat.com>
Reviewed-by: Antoni Segura Puimedon <asegurap at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
A pylintrc
M vdsm/API.py
2 files changed, 321 insertions(+), 0 deletions(-)
Approvals:
Sandro Bonazzola: Looks good to me, but someone else must approve
Giuseppe Vallarelli: Verified
Antoni Segura Puimedon: Looks good to me, but someone else must approve
Dan Kenigsberg: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/17772
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia41d2bf225e0b5e47a0e2718b906fbd44f5a54e9
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Giuseppe Vallarelli <gvallare at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl 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: Giuseppe Vallarelli <gvallare at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the vdsm-patches
mailing list