Add and run isort & codespell against python files

.github/workflows/bin/license

@@ -9,8 +9,8 @@ from __future__ import print_function
 
 import os
 import re
-from collections import defaultdict
 import sys
+from collections import defaultdict
 
 #: SPDX license id must appear in the first <license_lines> lines of a file
 license_lines = 7

.github/workflows/requirements/style.txt

@@ -1,2 +1,4 @@
 black==23.12.1
 flake8==7.0.0
+isort==5.13.2
+codespell==2.2.6

.github/workflows/style.yml

@@ -21,7 +21,9 @@ jobs:
         run: |
           pip install -r .github/workflows/requirements/style.txt
 
-      - name: Lint and Format Check with Flake8 and Black
+      - name: Lint and Format Check
         run: |
           black --diff --check .
+          codespell
+          isort
           flake8

bin/benchpark

@@ -6,11 +6,11 @@
 # SPDX-License-Identifier: Apache-2.0
 
 import argparse
-import subprocess
-import pathlib
 import os
-import shutil
+import pathlib
 import shlex
+import shutil
+import subprocess
 import sys
 
 DEBUG = False

docs/generate-sys-defs-list.py

@@ -1,8 +1,9 @@
 #!/usr/bin/env python3
 
-import yaml
 import glob
+
 import pandas as pd
+import yaml
 
 
 def main():

pyproject.toml

@@ -6,6 +6,7 @@ requires-python = ">=3.8"
 
 [tool.black]
 line-length = 88
+color = true
 target-version = ['py38', 'py39', 'py310', 'py311']
 include = '\.pyi?$|bin\/benchpark'
 exclude = '''
@@ -23,3 +24,12 @@ exclude = '''
   | repo
 )/
 '''
+
+[tool.isort]
+profile = "black"
+skip_gitignore = true
+color_output = true
+
+[tool.codespell]
+skip = '*.rst'
+ignore-words-list = 'fom'

repo/amg2023/application.py

@@ -3,9 +3,10 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 
+import sys
+
 from ramble.appkit import *
 
-import sys
 
 class Amg2023(SpackApplication):
     """AMG2023 benchmark"""

repo/cray-mpich/package.py

@@ -6,6 +6,7 @@
 from spack.package import *
 from spack.pkg.builtin.cray_mpich import CrayMpich as BuiltinCM
 
+
 class CrayMpich(BuiltinCM):
 
     variant("gtl", default=False, description="enable GPU-aware mode")

repo/cublas/package.py

@@ -5,6 +5,7 @@
 
 from spack.package import *
 
+
 class Cublas(Package):
 
     provides("blas")

repo/hypre/package.py

@@ -3,10 +3,11 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 
+import os
+
 from spack.package import *
 from spack.pkg.builtin.hypre import Hypre as BuiltinHypre
 
-import os
 
 class Hypre(BuiltinHypre):
     requires("+rocm", when="^rocblas")

repo/lapack-xl/package.py

@@ -5,6 +5,7 @@
 
 from spack.package import *
 
+
 class LapackXl(Package):
 
     provides("lapack")

repo/lbann/application.py

@@ -3,9 +3,10 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 
+import sys
+
 from ramble.appkit import *
 
-import sys
 
 class LBANN(SpackApplication):
     """LBANN benchmark"""

repo/raja-perf/application.py

@@ -3,9 +3,10 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 
+import sys
+
 from ramble.appkit import *
 
-import sys
 
 class RajaPerf(SpackApplication):
     """RAJA Performance suite"""

repo/raja-perf/package.py

@@ -3,16 +3,16 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 
-import os
-import socket
 import glob
+import os
 import re
-
+import socket
 from os import environ as env
 from os.path import join as pjoin
 
 from spack import *
 
+
 def spec_uses_toolchain(spec):
     gcc_toolchain_regex = re.compile(".*gcc-toolchain.*")
     using_toolchain = list(filter(gcc_toolchain_regex.match, spec.compiler_flags["cxxflags"]))

repo/rocblas/package.py

@@ -6,6 +6,7 @@
 from spack.package import *
 from spack.pkg.builtin.rocblas import Rocblas as BuiltinRocblas
 
+
 class Rocblas(BuiltinRocblas):
 
     provides("blas")

repo/rocsolver/package.py

@@ -6,6 +6,7 @@
 from spack.package import *
 from spack.pkg.builtin.rocsolver import Rocsolver as BuiltinRocsolver
 
+
 class Rocsolver(BuiltinRocsolver):
 
     provides("lapack")

repo/saxpy/application.py

@@ -3,9 +3,10 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 
+import sys
+
 from ramble.appkit import *
 
-import sys
 
 class Saxpy(SpackApplication):
     """saxpy benchmark"""

repo/saxpy/package.py

@@ -3,10 +3,12 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 
-from spack.package import *
 import os
 import shutil
+
 import spack.repo
+from spack.package import *
+
 
 class Saxpy(CMakePackage, CudaPackage, ROCmPackage):
     """Test saxpy problem."""