Skip to content

fix: enable mypy strict mode in all test files#585

Merged
mergify[bot] merged 1 commit intomainfrom
devs/jd/fix/mypy-strict-tests/I872c723b28364cf30c546c767ad665336ef62df4
Feb 19, 2026
Merged

fix: enable mypy strict mode in all test files#585
mergify[bot] merged 1 commit intomainfrom
devs/jd/fix/mypy-strict-tests/I872c723b28364cf30c546c767ad665336ef62df4

Conversation

@jd
Copy link
Copy Markdown
Owner

@jd jd commented Feb 19, 2026

Remove blanket mypy disable comments from test_tenacity.py,
test_asyncio.py, test_tornado.py, and test_after.py. Add type
annotations to all test methods, helper functions, and inner
functions so they pass mypy strict without suppressions.

Targeted type: ignore comments are used only for intentional
type mismatches in tests (e.g. passing None where BaseRetrying
is expected) and dynamically-added attributes on decorated
functions (.statistics, .retry, .retry_with).

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

Remove blanket mypy disable comments from test_tenacity.py,
test_asyncio.py, test_tornado.py, and test_after.py. Add type
annotations to all test methods, helper functions, and inner
functions so they pass mypy strict without suppressions.

Targeted type: ignore comments are used only for intentional
type mismatches in tests (e.g. passing None where BaseRetrying
is expected) and dynamically-added attributes on decorated
functions (.statistics, .retry, .retry_with).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Change-Id: I872c723b28364cf30c546c767ad665336ef62df4
@jd jd marked this pull request as ready for review February 19, 2026 09:07
@mergify mergify Bot merged commit 89c5735 into main Feb 19, 2026
9 checks passed
@mergify mergify Bot deleted the devs/jd/fix/mypy-strict-tests/I872c723b28364cf30c546c767ad665336ef62df4 branch February 19, 2026 09:08
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Feb 19, 2026

Merge Queue Status

Rule: default


  • Entered queue2026-02-19 09:08 UTC
  • Checks passed · in-place
  • Merged2026-02-19 09:08 UTC · at aa5da5ad2000b29b099cf92fb01e2dc3779691f2

This pull request spent 27 seconds in the queue, including 4 seconds running CI.

Required conditions to merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant