FIX: Deal with some "pylint: disable=" comments 09/23809/1
authorPeter Mikus <pmikus@cisco.com>
Thu, 5 Dec 2019 07:08:11 +0000 (07:08 +0000)
committerPeter Mikus <pmikus@cisco.com>
Thu, 5 Dec 2019 07:08:43 +0000 (07:08 +0000)
Same rules for all

Signed-off-by: Peter Mikus <pmikus@cisco.com>
Change-Id: Iaf6b6313750d19132e2f2886e676100878719690

resources/libraries/python/PapiExecutor.py

index 5a79a60..cbb3e28 100644 (file)
@@ -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.