docs: Trim restating comments and consolidate parity allowlist notes

Author: jsonbailey

ldclient/async_client.py

@@ -183,7 +183,6 @@ async def close(self, close_timeout: float = 2.0) -> None:
                 except Exception as e:
                     log.warning("Error closing AsyncLDClient components: %s", e)
 
-            # Close HTTP session
             if self._session is not None:
                 try:
                     await self._session.close()

ldclient/testing/test_async_client.py

@@ -102,7 +102,6 @@ async def test_close_is_idempotent():
     client = AsyncLDClient(_offline_config())
     await client.start()
     await client.close()
-    # Second close should be a no-op
     await client.close()
 
 
@@ -111,7 +110,6 @@ async def test_context_manager():
     """async with AsyncLDClient(config) as client: starts and closes the client."""
     async with AsyncLDClient(_offline_config()) as client:
         assert client.is_initialized()
-    # After exiting, closed flag should be set
     assert client._closed is True
 
 

ldclient/testing/test_sync_async_parity.py

@@ -28,24 +28,20 @@ def _public_surface(cls) -> set:
 
 
 # (sync_cls, async_cls, sync_only, async_only)
-# The allowlists document intentionally one-sided public members. Keep them
-# small and justified -- every entry is a place the two APIs deliberately differ.
+# The allowlists document intentionally one-sided public members, either
+# sync-only or async-only. Keep them small and justified -- every entry is a
+# place the two APIs deliberately differ.
 PAIRS = [
     pytest.param(
         LDClient, AsyncLDClient,
-        {"postfork"},   # sync-only: os.fork() recovery hook (no async equivalent)
-        # async-only: explicit `await start()` lifecycle, and flush_and_wait
-        # (the sync client only offers fire-and-forget flush()).
+        {"postfork"},
         {"start", "flush_and_wait"},
         id="client",
     ),
     pytest.param(
         InMemoryFeatureStore, AsyncInMemoryFeatureStore,
         set(),
-        {"close"},      # async-only: the async FeatureStore interface declares
-                        # `async def close()` for resource teardown; the sync
-                        # FeatureStore has no close() (it is hasattr-guarded at
-                        # every call site).
+        {"close"},
         id="feature_store",
     ),
     pytest.param(Evaluator, AsyncEvaluator, set(), set(), id="evaluator"),