matrix-org / matrix-org/matrix-python-common

Import twisted stubs from sydent

オープン
#12 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
Python
スター
4
フォーク
7
PR マージ指標
30日以内にマージされた PR はありません

説明

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).

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず、Sydent の stubs ディレクトリにある stubs と、このリポジトリの packaging 設定を確認します。コードを重複させずに、Twisted stubs をここで提供しつつ Sydent から import 可能にする方法を検討します。完了の条件は、共有 stubs をこのライブラリと Sydent の両方から利用でき、既存の型付き呼び出しで現在の workaround が不要になることです。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
python
領域
developer-experience, tooling
issue の種類
機能追加
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
25/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。