Skip to content

Fix mypy errors with Twisted 25.5 #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: anoa/codex/twisted_25_5_0
Choose a base branch
from

Conversation

anoadragon453-codex
Copy link
Owner

Summary

  • fix request key types for ReplicationAgent
  • update ProxyAgent reactor types and pool keys
  • drop obsolete ignores and fix custom reactor resolver

Testing

  • ruff check synapse/http/replicationagent.py synapse/http/proxyagent.py synapse/http/server.py tests/server.py tests/replication/_base.py tests/test_server.py
  • poetry run mypy
  • HTTP_PROXY= HTTPS_PROXY= http_proxy= https_proxy= no_proxy= NO_PROXY= poetry run trial tests.http.federation.test_matrix_federation_agent.MatrixFederationAgentTests.test_get

https://chatgpt.com/codex/tasks/task_e_68554ad84e2083318845367628e25869

@anoadragon453 anoadragon453 force-pushed the codex/fix-unit-tests-after-twisted-upgrade branch from 8b32221 to c4b5979 Compare June 20, 2025 17:03
anoadragon453 and others added 3 commits June 20, 2025 18:06
Otherwise they end up overrunning Twisted’s AMP protocol message size and raising a `TooLong` error.
@anoadragon453 anoadragon453 force-pushed the codex/fix-unit-tests-after-twisted-upgrade branch from c4b5979 to df108f6 Compare June 20, 2025 17:39
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.

2 participants