From 4bddfe8cb71f01a9da6b8c640e4af8436b9f0a4f Mon Sep 17 00:00:00 2001 From: Peter Mikus Date: Thu, 5 Dec 2019 07:08:11 +0000 Subject: [PATCH] FIX: Deal with some "pylint: disable=" comments Same rules for all Signed-off-by: Peter Mikus Change-Id: Iaf6b6313750d19132e2f2886e676100878719690 --- resources/libraries/python/PapiExecutor.py | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/resources/libraries/python/PapiExecutor.py b/resources/libraries/python/PapiExecutor.py index 5a79a60903..cbb3e28603 100644 --- a/resources/libraries/python/PapiExecutor.py +++ b/resources/libraries/python/PapiExecutor.py @@ -196,12 +196,9 @@ class PapiSocketExecutor: # Package path has to be one level above the vpp_papi directory. package_path = package_path.rsplit(u"/", 1)[0] sys.path.append(package_path) - # Only now, interpreter has a chance to locate the code to import. - # That means the import statement here is in the correct place. - # No refactor allows the import to be moved to where pylint wants, - # and pylint does not execute the logic for locating the code, - # so, dear pylint, please ignore these offences. - # pylint: disable=import-outside-toplevel, import-error + # TODO: Pylint says import-outside-toplevel and import-error. + # It is right, we should refactor the code and move initialization + # of package outside. from vpp_papi.vpp_papi import VPPApiClient as vpp_class vpp_class.apidir = api_json_directory # We need to create instance before removing from sys.path. -- 2.16.6