Skip to content

Commit 30114d3

Browse files
committed
fix: use correct pyright ignore syntax for optional logfire import
Changed from 'pyright: ignore' to 'pyright: ignore[reportMissingImports]' to properly suppress the import error for the optional logfire dependency. Logfire is only imported when cloud_mode is enabled and is wrapped in a try/except ImportError block, making it truly optional. Signed-off-by: phernandez <paul@basicmachines.co>
1 parent 9df359e commit 30114d3

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/basic_memory/api/app.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ async def lifespan(app: FastAPI): # pragma: no cover
3434
# Instrument httpx client for distributed tracing when in cloud mode
3535
if app_config.cloud_mode_enabled:
3636
try:
37-
import logfire # pyright: ignore
37+
import logfire # pyright: ignore[reportMissingImports]
3838
from basic_memory.mcp.async_client import client as httpx_client
3939

4040
logger.info("Cloud mode enabled - instrumenting httpx client for distributed tracing")

0 commit comments

Comments
 (0)