Merge pull request #164 from brews/ruff_refresh

Ruff refresh

.github/workflows/pythonpackage.yaml

@@ -34,7 +34,7 @@ jobs:
           pip install -r requirements.txt
       - name: Format check with ruff
         run: |
-          ruff check .
+          ruff check . --diff
 
   test:
     runs-on: ubuntu-latest

HISTORY.rst

@@ -6,7 +6,7 @@ History
 Unreleased
 ----------
 
-*
+* Minor code format cleanup, ruff config update.
 
 
 0.2.0 (2023-09-01)

pyproject.toml

@@ -53,6 +53,8 @@ exclude = [
     ".eggs",
     "docs",
 ]
+
+[tool.ruff.lint]
 # E402: module level import not at top of file
 ignore = [
     "E402",
@@ -65,4 +67,4 @@ select = [
     "W",
     # Pyupgrade
     "UP",
-]
+]

requirements.txt

@@ -2,4 +2,4 @@
 numpy==1.26.4
 pytest==8.1.1
 pytest-cov==5.0.0
-ruff==0.3.0
+ruff==0.3.5

src/parameterize_jobs/parameterize_jobs.py

@@ -50,9 +50,10 @@ def __getitem__(self, idx):
         elif isinstance(idx, int):
 
             if (idx >= len(self)) or (idx < -1 * len(self)):
+                name = self.__class__.__name__
+                n = len(self)
                 raise KeyError(
-                    'index {} out of bounds for {} with length {}'
-                    .format(idx, self.__class__.__name__, len(self)))
+                    f'index {idx} out of bounds for {name} with length {n}')
 
             lens = list(map(len, self._sets.values()))
             cplens = list(_cumprod(list(
@@ -137,9 +138,10 @@ def __len__(self):
 
     def __getitem__(self, idx):
         if (idx >= len(self)) or (idx < -1 * len(self)):
+            name = self.__class__.__name__
+            n = len(self)
             raise KeyError(
-                'index {} out of bounds for {} with length {}'
-                .format(idx, self.__class__.__name__, len(self)))
+                f'index {idx} out of bounds for {name} with length {n}')
 
         idx = idx % len(self)