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