Fixup optional args

src/con_duct/suite/ls.py

@@ -55,7 +55,9 @@
 MINIMUM_SCHEMA_VERSION: str = "0.2.0"
 
 
-def load_duct_runs(info_files: List[str], eval_filter: str) -> List[Dict[str, Any]]:
+def load_duct_runs(
+    info_files: List[str], eval_filter: Optional[str] = None
+) -> List[Dict[str, Any]]:
     loaded: List[Dict[str, Any]] = []
     for info_file in info_files:
         with open(info_file) as file:
@@ -70,7 +72,7 @@ def load_duct_runs(info_files: List[str], eval_filter: str) -> List[Dict[str, An
                         f"is below minimum schema version {MINIMUM_SCHEMA_VERSION}."
                     )
                     continue
-                if eval_filter and not eval(
+                if eval_filter is not None and not eval(
                     eval_filter, _flatten_dict(this), dict(re=re)
                 ):
                     continue

test/test_suite.py

@@ -4,7 +4,7 @@
 import json
 import os
 import tempfile
-from typing import Any
+from typing import Any, Optional
 import unittest
 from unittest.mock import MagicMock, mock_open, patch
 import pytest
@@ -184,7 +184,7 @@ def tearDown(self) -> None:
         self.temp_dir.cleanup()
 
     def _run_ls(
-        self, paths: list[str], fmt: str, args: argparse.Namespace = None
+        self, paths: list[str], fmt: str, args: Optional[argparse.Namespace] = None
     ) -> str:
         """Helper function to run ls() and capture stdout."""
         if args is None: