Replace mypy-protobuf with protobuf pyi_out
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 5.1k
- Fork
- 2.1k
- Merge medio
- 1g 19h
- PR unite (30g)
- 82
Descrizione
I noticed that pyi_out is a lot cleaner and more readable that mypy-protobuf. Since pyi generation is part of protobuf it is probably prefered instead of unofficial mypy_protobuf.
And there is also one interesting sideeffect that enums constants are not compatible between themselves exposing couple of bugs in our code.
For example:
enum EnumA {
A_A = 0;
A_B = 1;
}
enum EnumB {
B_A = 0;
B_B = 1;
}
message Msg {
EnumA name = 1;
}
mypy is not bothered that name can also be used with B_A and B_B. But it works with pyi_out generated code.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia individuando la configurazione del repository o gli script che invocano mypy-protobuf e analizza come vengono utilizzati gli stub Python generati. Confronta l'output attuale con il comportamento di pyi_out di protobuf, quindi verifica che la sostituzione preservi gli stub previsti e impedisca l'accettazione di costanti enum incompatibili.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- tooling
- Tipo di issue
- Refactoring
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100