Remove unnecessary assignment before return statement

nornir/init_nornir.py

@@ -35,8 +35,7 @@ def load_runner(
 ) -> RunnerPlugin:
     RunnersPluginRegister.auto_register()
     runner_plugin = RunnersPluginRegister.get_plugin(config.runner.plugin)
-    runner = runner_plugin(**config.runner.options)
-    return runner
+    return runner_plugin(**config.runner.options)
 
 
 def InitNornir(

pyproject.toml

@@ -169,7 +169,6 @@ ignore = [
     "PTH100",  # `os.path.abspath()` should be replaced by `Path.resolve()`
     "PTH120",  # `os.path.dirname()` should be replaced by `Path.parent`
     "PTH123",  # `open()` should be replaced by `Path.open()`
-    "RET504",  # Unnecessary assignment before `return` statement
     "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`

tests/conftest.py

@@ -43,7 +43,7 @@ def get_defaults():
         with open(defaults_file, "r") as f:
             defaults_dict = yml.load(f)
 
-            defaults = Defaults(
+            return Defaults(
                 hostname=defaults_dict.get("hostname"),
                 port=defaults_dict.get("port"),
                 username=defaults_dict.get("username"),
@@ -55,8 +55,6 @@ def get_defaults():
                 ),
             )
 
-        return defaults
-
     def get_inventory_element(typ, data, name, defaults):
         return typ(
             name=name,
@@ -124,8 +122,7 @@ def inv(request):
 @pytest.fixture(scope="session", autouse=True)
 def nornir(request):
     """Initializes nornir"""
-    nr = Nornir(inventory=inventory_from_yaml(), runner=SerialRunner(), data=global_data)
-    return nr
+    return Nornir(inventory=inventory_from_yaml(), runner=SerialRunner(), data=global_data)
 
 
 @pytest.fixture(scope="function", autouse=True)