✨ Refactor: Deprecate connect_to_db function and recommend postgres_lifespan for database connection management

Author: pcrespov

packages/service-library/src/servicelib/db_async_engine.py

@@ -22,6 +22,7 @@ async def connect_to_db(app: FastAPI, settings: PostgresSettings) -> None:
         "The 'connect_to_db' function is deprecated and will be removed in a future release. "
         "Please use 'postgres_lifespan' instead for managing the database connection lifecycle.",
         DeprecationWarning,
+        stacklevel=2,
     )
     with log_context(
         _logger, logging.DEBUG, f"connection to db {settings.dsn_with_async_sqlalchemy}"

packages/service-library/src/servicelib/fastapi/db_asyncpg_engine.py

@@ -1,4 +1,5 @@
 import logging
+import warnings
 
 from fastapi import FastAPI
 from settings_library.postgres import PostgresSettings
@@ -14,6 +15,13 @@
 
 
 async def connect_to_db(app: FastAPI, settings: PostgresSettings) -> None:
+    warnings.warn(
+        "The 'connect_to_db' function is deprecated and will be removed in a future release. "
+        "Please use 'postgres_lifespan' instead for managing the database connection lifecycle.",
+        DeprecationWarning,
+        stacklevel=2,
+    )
+
     with log_context(
         _logger,
         logging.DEBUG,