From 3d204b0964a9f8d51b8c97429caf2eae89504050 Mon Sep 17 00:00:00 2001 From: TiM Date: Fri, 16 Oct 2026 17:45:40 +1300 Subject: [PATCH] Chore: Remove an unused `ty: ignore` in the AppsFlyer test - `poe lint` passes on `main` again, so the integration jobs get past lint to the test suite. ty 0.0.85 reports the `# ty: ignore[invalid-context-manager]` at `tests/warehouse/saas/test_appsflyer.py:200` as `unused-ignore-comment`, and `poe lint` fails on that warning. --- tests/warehouse/saas/test_appsflyer.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/warehouse/saas/test_appsflyer.py b/tests/warehouse/saas/test_appsflyer.py index 8182a54af..7e84c9d55 100644 --- a/tests/warehouse/saas/test_appsflyer.py +++ b/tests/warehouse/saas/test_appsflyer.py @@ -197,7 +197,7 @@ def test_exclude_metrics_for_date_range(): to_date = "2024-01-11" now = "2024-01-12" - with pendulum.travel_to(cast(pendulum.DateTime, pendulum.parse(now))): # ty: ignore[invalid-context-manager] + with pendulum.travel_to(cast(pendulum.DateTime, pendulum.parse(now))): excluded_metrics = exclude_metrics_for_date_range(metrics, from_date, to_date) assert excluded_metrics == [ "cohort_day_1_revenue_per_user",