chore: add ruff TRY

Signed-off-by: Henry Schreiner <[email protected]>

Author: henryiii

pyproject.toml

@@ -162,6 +162,7 @@ extend-select = [
     "RUF", # Ruff-specific
     "SIM", # flake8-simplify
     "T20", # flake8-print
+    "TRY", # tryceratops
     "UP", # pyupgrade
     "YTT", # flake8-2020
     "EXE", # flake8-executable

src/mplhep/_dev.py

@@ -102,20 +102,22 @@ def _run_command_with_confirmation(
 
     def _run_command(self, cmd: list[str], cwd: Path | None = None) -> bool:
         """Run a command and return True if successful."""
+        self._print_header(f"Running: {' '.join(cmd)}")
+        separator = 3 * ("=" * self._get_terminal_width() + "\n")
+        print(separator)
+
         try:
-            self._print_header(f"Running: {' '.join(cmd)}")
-            separator = 3 * ("=" * self._get_terminal_width() + "\n")
-            print(separator)
             result = subprocess.run(cmd, cwd=cwd or self.project_root, check=True)
-            print(separator)
-            return result.returncode == 0
         except subprocess.CalledProcessError as e:
             self._print_error(f"Command failed with exit code {e.returncode}")
             return False
         except FileNotFoundError:
             self._print_error(f"Command not found: {cmd[0]}")
             return False
 
+        print(separator)
+        return result.returncode == 0
+
     def _show_summary(self, items: list[Path], title: str) -> None:
         """Show a formatted summary of items."""
         if not items:
@@ -856,13 +858,14 @@ def _check_tool_available(self, tool_name: str, check_cmd: list[str]) -> bool:
             result = subprocess.run(
                 check_cmd, capture_output=True, text=True, check=True
             )
-            self._print_success(f"{tool_name} version: {result.stdout.strip()}")
-            return True
         except (subprocess.CalledProcessError, FileNotFoundError):
             self._print_error(f"{tool_name} not found!")
             self._print_warning(f"Please install {tool_name} to use this feature")
             return False
 
+        self._print_success(f"{tool_name} version: {result.stdout.strip()}")
+        return True
+
     def _handle_pytest_results_cleanup(self) -> None:
         """Handle cleanup of pytest_results directory."""
         pytest_results = self.project_root / "pytest_results"

tests/conftest.py

@@ -16,13 +16,13 @@ def _has_latex():
             capture_output=True,
             timeout=5,
         )
-        return True
     except (
         subprocess.CalledProcessError,
         subprocess.TimeoutExpired,
         FileNotFoundError,
     ):
         return False
+    return True
 
 
 def pytest_collection_modifyitems(config, items):  # noqa: ARG001