From e6d8d95c0a781649767e2b28660de7a50771c157 Mon Sep 17 00:00:00 2001
From: Taimoor Zaeem <taimoorzaeem@gmail.com>
Date: Mon, 18 May 2026 17:46:54 +0500
Subject: [PATCH] chore: remove unplanned TODO in Logger.hs

We decided against doing this in #4913, therefore removing it. With that,
also mentioning that control flow never reaches there to avoid confusion.

Signed-off-by: Taimoor Zaeem <taimoorzaeem@gmail.com>
---
 src/PostgREST/Logger.hs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/PostgREST/Logger.hs b/src/PostgREST/Logger.hs
index 3a128331e..b40585ca8 100644
--- a/src/PostgREST/Logger.hs
+++ b/src/PostgREST/Logger.hs
@@ -234,7 +234,7 @@ observationMessages = \case
   WarpServerObs txt ->
     pure $ "Warp server: " <> txt
   ResponseObs {} ->
-    mempty -- TODO this message is produced on observationLogger since it depends on Logger state. Merge observationMessages with observationLogger to clear this.
+    mempty -- Control flow never reaches here, the observation message is returned in observationLogger function
   where
     showMillis :: Double -> Text
     showMillis x = toS $ showFFloat (Just 1) x ""