From 68bfb4f61576d93a3855877f9790e3235a9a4380 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?L=C3=A9on=20Avic=20Simmons?= Date: Mon, 7 Sep 2026 05:15:32 -0400 Subject: [PATCH] docs: fix typo and copy-pasted docstrings in telemetry MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - telemetry/core/service.py: 'Supress' -> 'Suppress' - telemetry/interactive/prompts/identity.py: two docstrings copied from newsletter.py now describe the identity prompt Comments and docstrings only; no functional changes. Signed-off-by: Léon Avic Simmons --- src/tabpfn_common_utils/telemetry/core/service.py | 2 +- .../telemetry/interactive/prompts/identity.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/tabpfn_common_utils/telemetry/core/service.py b/src/tabpfn_common_utils/telemetry/core/service.py index 6e4ec88..3e40325 100644 --- a/src/tabpfn_common_utils/telemetry/core/service.py +++ b/src/tabpfn_common_utils/telemetry/core/service.py @@ -15,7 +15,7 @@ # Suppress PostHog and analytics-python logs (unless explicitly enabled) if os.getenv("TABPFN_ENABLE_TELEMETRY_LOGS", "0").lower() not in ("1", "true"): - # Supress warnings + # Suppress warnings for logger_name in ["posthog", "analytics", "posthog.analytics"]: logging.getLogger(logger_name).setLevel(logging.CRITICAL) diff --git a/src/tabpfn_common_utils/telemetry/interactive/prompts/identity.py b/src/tabpfn_common_utils/telemetry/interactive/prompts/identity.py index 4ef6b31..32c8d9e 100644 --- a/src/tabpfn_common_utils/telemetry/interactive/prompts/identity.py +++ b/src/tabpfn_common_utils/telemetry/interactive/prompts/identity.py @@ -65,13 +65,13 @@ def _parse(raw: str) -> tuple[Optional[Outcome], Optional[Dict[str, Any]]]: def _should_prompt() -> bool: """ - Check if the user should be prompted to subscribe to the newsletter. + Check if the user should be prompted to share anonymous usage analytics. """ return False def _on_done(res: PromptResult) -> None: - """Done callback for newsletter prompt. + """Done callback for identity prompt. Args: res: The prompt result.