Remove obsolete noqa statements (#944)

nornir/core/__init__.py

@@ -144,7 +144,7 @@ def run(
 
         raise_on_error = (
             raise_on_error if raise_on_error is not None else self.config.core.raise_on_error
-        )  # noqa
+        )
         if raise_on_error:
             result.raise_on_error()
         else:

nornir/core/configuration.py

@@ -149,7 +149,7 @@ def configure(self) -> None:
                 "This can lead to unexpected logging results. "
                 "Please set logging.enabled config to False "
                 "to disable automatic Nornir logging configuration. Refer to "
-                "https://nornir.readthedocs.io/en/stable/configuration/index.html#logging"  # noqa
+                "https://nornir.readthedocs.io/en/stable/configuration/index.html#logging"
             )
             warnings.warn(msg, ConflictingConfigurationWarning)
 

nornir/core/exceptions.py

@@ -2,7 +2,7 @@
 
 if TYPE_CHECKING:
     from nornir.core.connection import Connection
-    from nornir.core.result import AggregatedResult, MultiResult, Result  # noqa
+    from nornir.core.result import AggregatedResult, MultiResult, Result
     from nornir.core.tasks import Task
 
 

pyproject.toml

@@ -157,7 +157,6 @@ ignore = [
     "RSE102",  # Unnecessary parentheses on raised exception
     "RUF001",  # String contains ambiguous `–` (EN DASH). Did you mean `-` (HYPHEN-MINUS)?
     "RUF012",  # Mutable class attributes should be annotated with `typing.ClassVar`
-    "RUF100",  # Unused blanket `noqa` directive
     "S701",    # By default, jinja2 sets `autoescape` to `False`. Consider using `autoescape=True` or the `select_autoescape` function to mitigate XSS vulnerabilities.
     "SLF001",  # Private member accessed
     "SIM108",  # Use ternary operator `config = Config.from_file(config_file, **kwargs) if config_file else Config.from_dict(**kwargs)` instead of `if`-`else`-block