More style adjustments

sandialabs · Nov 30, 2023 · 63f3791 · 63f3791
1 parent 9cd68e4
commit 63f3791
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 20 deletions.
diff --git a/manager/__init__.py b/manager/__init__.py
@@ -7,7 +7,6 @@
 
 import importlib.util
 import os
-import sys
 
 import spack.util.spack_yaml as syaml
 
@@ -68,7 +67,6 @@ def _machine_detector(self):
 
     def _populate_machines(self):
         self.machines = []
-        machine_path = os.path.join(self.root, "configs")
         machine_dirs = list(os.scandir(os.path.join(self.root, "configs")))
         for machine in machine_dirs:
             self.machines.append(machine.name)

diff --git a/manager/cmd/manager.py b/manager/cmd/manager.py
@@ -13,14 +13,14 @@
 sys.path.append(module_dir)
 
 # import manager_cmds.create_dev_env
-import manager_cmds.create_env
+import manager_cmds.create_env  # noqa E402
 
-# import manager_cmds.develop
-# import manager_cmds.external
-import manager_cmds.find_machine
+# import manager_cmds.develop # noqa E402
+# import manager_cmds.external # noqa E402
+import manager_cmds.find_machine  # noqa E402
 
-# import manager_cmds.pin
-# import manager_cmds.snapshot
+# import manager_cmds.pin # noqa E402
+# import manager_cmds.snapshot # noqa E402
 
 if sys.version_info[0] < 3:
     print("spack-manager commands only support python 3")

diff --git a/manager/manager_cmds/create_env.py b/manager/manager_cmds/create_env.py
@@ -50,14 +50,14 @@ def create_env(parser, args):
     if not args.yaml:
         manifest.set_config_value("concretizer", "unify", True)
 
+    msg = (
+        "Specified machine {m} was not found. "
+        "To see registered machines run `spack manager find-machine --list`"
+    )
     if args.machine is not None:
         project = machine_defined(args.machine)
         if not project:
-            tty.error(
-                "Specified machine {m} was not found. To see registered machines run `spack manager find-machine --list`".format(
-                    m=args.machine
-                )
-            )
+            tty.error(msg.format(m=args.machine))
             sys.exit(1)
         else:
             machine = args.machine
@@ -66,11 +66,7 @@ def create_env(parser, args):
         print(find_machine())
         project, machine = find_machine()
         if machine == "NOT-FOUND":
-            tty.warn(
-                "Specified machine {m} was not found. To see registered machines run `spack manager find-machine --list`".format(
-                    m=args.machine
-                )
-            )
+            tty.warn(msg.format(m=args.machine))
 
     if args.spec:
         spec_list = spack.cmd.parse_specs(args.spec)

diff --git a/manager/manager_cmds/find_machine.py b/manager/manager_cmds/find_machine.py
@@ -5,8 +5,6 @@
 # This software is released under the BSD 3-clause license. See LICENSE file
 # for more details.
 
-import os
-import socket
 import sys
 
 import manager