danielgtaylor / danielgtaylor/python-betterproto

Suggestion: use abstract base class and abstractmethod for ServiceBase and/or Stub

オープン
#376 コメント 1 件 リアクション 1 件 担当者 0 名 GitHub で見る
enhancement good first issue small
主要言語
Python
スター
1.8k
フォーク
234
PR マージ指標
30日以内にマージされた PR はありません

説明

python-betterprot is awesome.
However, users may forget to implement some service-methods declared in proto file, which cause no error on the server-side.
`@abstractmethod` may give user some hints to realize it.
One question: why user should implement method `echo_stream` in EchoService while the related rpc method named `EchoStream` in proto file?

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

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

調査の方向性

まず、生成された ServiceBase と Stub の定義、および proto RPC 名が Python のメソッド名になる仕組みを調べます。抽象メソッドに意図されている動作と命名を特定し、その後、存在しないサービスメソッドが明確に失敗すること、また EchoStream が一貫して echo_stream に対応付けられることを確認します。

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

評価

技術スタック
grpc, python
領域
backend-api-design
issue の種類
機能追加
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
25/100

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

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