chore: fix lint

nodejs · Jun 4, 2024 · e957453 · e957453
1 parent 4ddd247
commit e957453
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 9 deletions.
diff --git a/.github/workflows/lint_python.yml b/.github/workflows/lint_python.yml
@@ -15,9 +15,8 @@ jobs:
       - run: pip install --upgrade pip setuptools wheel
       - run: pip install --editable ".[dev]"
       - run: codespell .
-      - run: ruff --output-format=github .
+      - run: ruff check --output-format=github .
       - run: black --check .
       - run: mypy .
       - run: pytest -vv .
       - run: pytest --doctest-modules .
-      - run: safety check
diff --git a/pyproject.toml b/pyproject.toml
@@ -34,7 +34,6 @@ dev = [
     "pytest",
     "pytest-clarity",
     "ruff",
-    "safety",
 ]
 
 [project.scripts]

diff --git a/tap2junit/tap13.py b/tap2junit/tap13.py
@@ -191,16 +191,16 @@ def _parse(self, source):
                         t_attrs["id"] = self.__tests_counter[-1]
                     t_attrs["id"] = int(t_attrs["id"])
                     if t_attrs["id"] < self.__tests_counter[-1]:
-                        raise ValueError("Descending test id on line: %r" % line)
+                        raise ValueError(f"Descending test id on line: {line!r}")
                     # according to TAP13 specs, missing tests must be handled as
                     # 'not ok' so here we add the missing tests in sequence
                     while t_attrs["id"] > self.__tests_counter[-1]:
+                        counter = self.__tests_counter[-1]
                         self.tests.append(
                             Test(
                                 "not ok",
                                 self.__tests_counter[-1],
-                                comment="DIAG: Test %s not present"
-                                % self.__tests_counter[-1],
+                                comment=f"DIAG: Test {counter} not present",
                             )
                         )
                         self.__tests_counter[-1] += 1
@@ -210,9 +210,7 @@ def _parse(self, source):
                         # according to TAP13 specs, everything after this is an
                         # explanation of why testing must be stopped
                         t.diagnostics = t.diagnostics or t.description
-                        t.description = (
-                            "Bail out for Test %s" % self.__tests_counter[-1]
-                        )
+                        t.description = f"Bail out for Test {self.__tests_counter[-1]}"
                     self.tests.append(t)
                     in_test = True
                     continue