danielgtaylor / danielgtaylor/python-betterproto

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

Ouverte
#376 1 commentaire 1 réaction 0 personnes assignées Voir sur GitHub
enhancement good first issue small
Langage dominant
Python
Étoiles
1.8k
Forks
234
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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?

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par inspecter les définitions générées de ServiceBase et Stub, ainsi que la façon dont les noms des RPC proto deviennent des noms de méthodes Python. Déterminez le comportement et le nommage attendus des méthodes abstraites, puis vérifiez que les méthodes de service manquantes échouent clairement et que EchoStream correspond systématiquement à echo_stream.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
grpc, python
Domaine
backend-api-design
Type d'issue
Fonctionnalité
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.