New parser deviates from old parser in edge cases with function type comments
Offen
Dieses Issue hat noch niemand übernommen.
bug
topic-parser
- Vorherrschende Sprache
- Python
- Sterne
- 20.6k
- Forks
- 3.3k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
This is a follow-up for https://github.com/python/mypy/pull/21823. Most notably we get a false negative in situations like this:
class A:
def f(self):
# type: () -> None
def g(x):
# type: () -> None # Error not detected: signature has too few parameters
pass
cc @JukkaL
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, das Beispiel mit der verschachtelten Klasse und Funktion aus dem issue zu reproduzieren, und vergleiche anschließend, wie der neue und der alte Parser Kommentare zu Funktionstypen behandeln. Die Arbeit ist abgeschlossen, wenn der neue Parser den fehlenden Parameter in der Signatur der inneren Funktion erkennt und damit in diesem Edge Case dem Verhalten des alten Parsers entspricht.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100