Skip to content

Commit e6d8d95

Browse files
taimoorzaeemwolfgangwalther
authored andcommitted
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>
1 parent 9d67feb commit e6d8d95

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/PostgREST/Logger.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,7 @@ observationMessages = \case
234234
WarpServerObs txt ->
235235
pure $ "Warp server: " <> txt
236236
ResponseObs {} ->
237-
mempty -- TODO this message is produced on observationLogger since it depends on Logger state. Merge observationMessages with observationLogger to clear this.
237+
mempty -- Control flow never reaches here, the observation message is returned in observationLogger function
238238
where
239239
showMillis :: Double -> Text
240240
showMillis x = toS $ showFFloat (Just 1) x ""

0 commit comments

Comments
 (0)