Skip to content

fix: resolve tornado mypy errors instead of suppressing them - #568

Merged
mergify[bot] merged 1 commit into
mainfrom
devs/jd/fix/tornado-mypy/I48134c99c48d7abe6734b34a79255fbcd3f39618
Feb 18, 2026
Merged

mergify[bot] merged 1 commit into
mainfrom
devs/jd/fix/tornado-mypy/I48134c99c48d7abe6734b34a79255fbcd3f39618

Conversation

@jd

@jd jd commented Feb 18, 2026

Copy link
Copy Markdown
Owner
  • Add class-level sleep type annotation to TornadoRetrying so the
    async sleep signature is accepted without ignore_errors
  • Remove stale type: ignore comments from tornadoweb.py and test_tornado.py
  • Drop the blanket ignore_errors mypy override for tornadoweb/test_tornado

- Add class-level sleep type annotation to TornadoRetrying so the
  async sleep signature is accepted without ignore_errors
- Remove stale type: ignore comments from tornadoweb.py and test_tornado.py
- Drop the blanket ignore_errors mypy override for tornadoweb/test_tornado

Change-Id: I48134c99c48d7abe6734b34a79255fbcd3f39618
@jd
jd marked this pull request as ready for review February 18, 2026 15:59
@mergify

mergify Bot commented Feb 18, 2026 •

Copy link
Copy Markdown
Contributor

Merge Queue Status

Rule: default


  • ✅ Entered queue — 2026-02-18 15:59 UTC
  • ✅ Checks passed · in-place
  • ✅ Merged — 2026-02-18 16:00 UTC · at e49034ce21ff7f0d81120c2e69726cfd88ad7fbf

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

Required conditions to merge

@mergify mergify Bot added the queued label Feb 18, 2026
@mergify
mergify Bot merged commit 1b2b4ef into main Feb 18, 2026
9 checks passed
@mergify
mergify Bot deleted the devs/jd/fix/tornado-mypy/I48134c99c48d7abe6734b34a79255fbcd3f39618 branch February 18, 2026 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant