From c2971871dd1380b88d58f29211a3e8478aca477a Mon Sep 17 00:00:00 2001 From: Martin Klozik Date: Mon, 12 Oct 2015 07:15:54 +0100 Subject: Add Pylint to VSPERF commit gate Initial pylint config file has been created to follow current code structure and design. A few issues have been fixed, so currently 56 of 67 files have rating 10/10. There are still issues to be fixed, mainly TODOs/FIXMEs and missing impelentation of abstract class members. Change-Id: Icf5b12c2150258f8ac657f8d50b47a697d1660a0 JIRA: VSPERF-91 Signed-off-by: Martin Klozik Reviewed-by: Maryam Tahhan Reviewed-by: Fatih Degirmenci Reviewed-by: Gene Snider Reviewed-by: Al Morton Reviewed-by: Tv Rao Reviewed-by: Brian Castelli --- docs/to-be-reorganized/quickstart.rst | 12 ++++++++++++ 1 file changed, 12 insertions(+) (limited to 'docs/to-be-reorganized/quickstart.rst') diff --git a/docs/to-be-reorganized/quickstart.rst b/docs/to-be-reorganized/quickstart.rst index c575be59..bb009fce 100644 --- a/docs/to-be-reorganized/quickstart.rst +++ b/docs/to-be-reorganized/quickstart.rst @@ -243,6 +243,18 @@ To run tests using Vanilla OVS: ./vsperf --conf-file +Code change verification by pylint +---------------------------------- +Every developer participating in VSPERF project should run +pylint before his python code is submitted for review. Project +specific configuration for pylint is available at 'pylint.rc'. + +Example of manual pylint invocation: + + .. code-block:: console + + pylint --rcfile ./pylint.rc ./vsperf + GOTCHAs: -------- -- cgit 1.2.3-korg