matrix-org / matrix-org/matrix-python-common
Import twisted stubs from sydent
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 4
- Forks
- 7
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Currently we have to force mypy to allow calling untyped functions from a typed context, because we call functions from twisted that aren't typed (at least as of #11). Sydent has a bunch of twisted stubs: https://github.com/matrix-org/sydent/tree/main/stubs - we should copy them to this repo and use them instead.
However it's not entirely clear to me how to do that in a way that wouldn't duplicate code, i.e. in a way we could import them in Sydent from this library (i.e. we want to move the stubs from Sydent to here, rather than just copying them over).
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die stubs im stubs-Verzeichnis von Sydent und die Packaging-Konfiguration dieses Repositorys zu überprüfen. Ermittle, wie die Twisted-stubs hier bereitgestellt werden können, während sie für Sydent importierbar bleiben, ohne duplizierten Code zu erzeugen. Erledigt bedeutet, dass die gemeinsamen stubs aus dieser Bibliothek und aus Sydent verwendet werden können und die bestehenden typisierten Aufrufe nicht mehr den aktuellen Workaround benötigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- developer-experience, tooling
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100