Fix pylint complaints after pylint release (kedro-org#1072)

Author: merelcht

.pre-commit-config.yaml

@@ -132,4 +132,4 @@ repos:
             language: system
             pass_filenames: false
             stages: [manual]
-            entry: pylint -j 4 --disable=missing-docstring,redefined-outer-name,no-self-use,invalid-name,protected-access,too-many-arguments,too-many-public-methods tests
+            entry: pylint -j 4 --disable=missing-docstring,redefined-outer-name,no-self-use,invalid-name,protected-access,too-many-arguments,too-many-public-methods,use-implicit-booleaness-not-comparison tests

features/steps/sh_run.py

@@ -20,7 +20,7 @@ def run(
         print_output: If True will print previously captured stdout.
             Default is False.
 
-        kwargs: Extra options to pass to subprocess.
+        **kwargs: Extra options to pass to subprocess.
 
     Example:
     ::

kedro/config/config.py

@@ -182,7 +182,7 @@ def get(self, *patterns: str) -> Dict[str, Any]:
         return them in the form of a config dictionary.
 
         Args:
-            patterns: Glob patterns to match. Files, which names match
+            *patterns: Glob patterns to match. Files, which names match
                 any of the specified patterns, will be processed.
 
         Raises:

kedro/config/templated_config.py

@@ -144,7 +144,7 @@ def get(self, *patterns: str) -> Dict[str, Any]:
         dictionary of replacement values obtained in the ``__init__`` method.
 
         Args:
-            patterns: Glob patterns to match. Files, which names match
+            *patterns: Glob patterns to match. Files, which names match
                 any of the specified patterns, will be processed.
 
         Returns:

kedro/framework/cli/cli.py

@@ -107,7 +107,7 @@ def get_project_context(
             Supported keys are "verbose" and "context", and it defaults to "context".
         project_path: Optional path to where the project root is to load the context.
             If omitted, the current working directory will be used.
-        kwargs: Optional custom arguments defined by users, which will be passed into
+        **kwargs: Optional custom arguments defined by users, which will be passed into
             the constructor of the projects KedroContext subclass.
 
     Returns:

kedro/framework/context/context.py

@@ -692,7 +692,7 @@ def load_context(project_path: Union[str, Path], **kwargs) -> KedroContext:
 
     Args:
         project_path: Path to the Kedro project.
-        kwargs: Optional kwargs for ``KedroContext`` class.
+        **kwargs: Optional kwargs for ``KedroContext`` class.
 
     Returns:
         Instance of ``KedroContext`` class defined in Kedro project.

kedro/framework/hooks/specs.py

@@ -135,6 +135,7 @@ def on_node_error(
 class PipelineSpecs:
     """Namespace that defines all specifications for a pipeline's lifecycle hooks."""
 
+    # pylint: disable=missing-param-doc
     @hook_spec
     def before_pipeline_run(
         self, run_params: Dict[str, Any], pipeline: Pipeline, catalog: DataCatalog

kedro/pipeline/node.py

@@ -319,7 +319,7 @@ def decorate(self, *decorators: Callable) -> "Node":
         new ``Node`` object, but with no changes to the function.
 
         Args:
-            decorators: Decorators to be applied on the node function.
+            *decorators: Decorators to be applied on the node function.
                 Decorators will be applied from right to left.
 
         Returns:

kedro/pipeline/pipeline.py

@@ -369,7 +369,7 @@ def only_nodes(self, *node_names: str) -> "Pipeline":
         nodes by name.
 
         Args:
-            node_names: One or more node names. The returned ``Pipeline``
+            *node_names: One or more node names. The returned ``Pipeline``
                 will only contain these nodes.
 
         Raises:
@@ -486,7 +486,7 @@ def only_nodes_with_inputs(self, *inputs: str) -> "Pipeline":
         matches to the fully-qualified name only (i.e. name@format).
 
         Args:
-            inputs: A list of inputs which should be used as a starting
+            *inputs: A list of inputs which should be used as a starting
                 point of the new ``Pipeline``.
 
         Raises:
@@ -512,7 +512,7 @@ def from_inputs(self, *inputs: str) -> "Pipeline":
         matches to the fully-qualified name only (i.e. name@format).
 
         Args:
-            inputs: A list of inputs which should be used as a starting point
+            *inputs: A list of inputs which should be used as a starting point
                 of the new ``Pipeline``
 
         Raises:
@@ -552,7 +552,7 @@ def only_nodes_with_outputs(self, *outputs: str) -> "Pipeline":
         to the fully-qualified name only (i.e. name@format).
 
         Args:
-            outputs: A list of outputs which should be the final outputs
+            *outputs: A list of outputs which should be the final outputs
                 of the new ``Pipeline``.
 
         Raises:
@@ -577,7 +577,7 @@ def to_outputs(self, *outputs: str) -> "Pipeline":
         to the fully-qualified name only (i.e. name@format).
 
         Args:
-            outputs: A list of outputs which should be the final outputs of
+            *outputs: A list of outputs which should be the final outputs of
                 the new ``Pipeline``.
 
         Raises:
@@ -613,7 +613,7 @@ def from_nodes(self, *node_names: str) -> "Pipeline":
         directly or transitively on the provided nodes.
 
         Args:
-            node_names: A list of node_names which should be used as a
+            *node_names: A list of node_names which should be used as a
                 starting point of the new ``Pipeline``.
         Raises:
             ValueError: Raised when any of the given names do not exist in the
@@ -634,7 +634,7 @@ def to_nodes(self, *node_names: str) -> "Pipeline":
         or transitively by the provided nodes.
 
         Args:
-            node_names: A list of node_names which should be used as an
+            *node_names: A list of node_names which should be used as an
                 end point of the new ``Pipeline``.
         Raises:
             ValueError: Raised when any of the given names do not exist in the
@@ -656,7 +656,7 @@ def only_nodes_with_tags(self, *tags: str) -> "Pipeline":
         are provided.
 
         Args:
-            tags: A list of node tags which should be used to lookup
+            *tags: A list of node tags which should be used to lookup
                 the nodes of the new ``Pipeline``.
         Returns:
             Pipeline: A new ``Pipeline`` object, containing a subset of the
@@ -673,7 +673,7 @@ def decorate(self, *decorators: Callable) -> "Pipeline":
         return a copy of the current ``Pipeline`` object.
 
         Args:
-            decorators: Decorators to be applied on all node functions in
+            *decorators: Decorators to be applied on all node functions in
                 the pipeline, always applied from right to left.
 
         Returns: