Add comment to say it should be removed

Author: DouweM

pydantic_evals/pydantic_evals/_utils.py

@@ -122,6 +122,7 @@ class LogfireNotConfiguredWarning(UserWarning):
 @contextmanager
 def logfire_span(*args: Any, **kwargs: Any) -> Generator[logfire_api.LogfireSpan, None, None]:
     """Create a Logfire span without warning if logfire is not configured."""
+    # TODO: Remove once Logfire has the ability to suppress this warning from non-user code
     with warnings.catch_warnings():
         warnings.filterwarnings('ignore', category=LogfireNotConfiguredWarning)
         with _logfire.span(*args, **kwargs) as span:

pydantic_graph/pydantic_graph/_utils.py

@@ -153,6 +153,7 @@ class LogfireNotConfiguredWarning(UserWarning):
 @contextmanager
 def logfire_span(*args: Any, **kwargs: Any) -> Generator[LogfireSpan, None, None]:
     """Create a Logfire span without warning if logfire is not configured."""
+    # TODO: Remove once Logfire has the ability to suppress this warning from non-user code
     with warnings.catch_warnings():
         warnings.filterwarnings('ignore', category=LogfireNotConfiguredWarning)
         with _logfire.span(*args, **kwargs) as span: