Update config to be compatible with pylint>=2.14 (#1595)

* Remove "bad-continuation" rule removed from pylint

Signed-off-by: Deepyaman Datta <[email protected]>

* Load "pylint.extensions.no_self_use" for `pylint>=2.14`

Signed-off-by: Deepyaman Datta <[email protected]>

* Ignore pylint warning from testing context manager

Signed-off-by: Deepyaman Datta <[email protected]>

* Ignore the "unnecessary" lambda assignment warning

Signed-off-by: Deepyaman Datta <[email protected]>

Author: deepyaman

pyproject.toml

@@ -14,12 +14,14 @@ default_section = "THIRDPARTY"
 [tool.pylint.master]
 ignore = "CVS"
 ignore-patterns = "kedro/templates/*"
-load-plugins = "pylint.extensions.docparams"
+load-plugins = [
+    "pylint.extensions.docparams",
+    "pylint.extensions.no_self_use"
+]
 unsafe-load-any-extension = false
 [tool.pylint.messages_control]
 disable = [
     "ungrouped-imports",
-    "bad-continuation",
     "duplicate-code"
 ]
 enable = ["useless-suppression"]

tests/extras/datasets/pandas/test_hdf_dataset.py

@@ -133,7 +133,10 @@ def test_thread_lock_usage(self, hdf_data_set, dummy_dataframe, mocker):
         mocked_lock.assert_not_called()
 
         hdf_data_set.save(dummy_dataframe)
-        calls = [mocker.call.__enter__(), mocker.call.__exit__(None, None, None)]
+        calls = [
+            mocker.call.__enter__(),  # pylint: disable=unnecessary-dunder-call
+            mocker.call.__exit__(None, None, None),
+        ]
         mocked_lock.assert_has_calls(calls)
 
         mocked_lock.reset_mock()

tests/pipeline/test_node.py

@@ -300,7 +300,7 @@ def dummy_func_args(**kwargs):
     return dummy_func_args, "A", "B"
 
 
-lambda_identity = lambda input1: input1  # noqa: disable=E731
+lambda_identity = lambda input1: input1  # noqa: disable=E731 # pylint: disable=C3001
 
 
 def lambda_inconsistent_input_size():