Add truthy-coroutine check
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 20.6k
- Forks
- 3.3k
- Ø Merge
- 1 T. 18 Std.
- Gemergte PRs (30 T.)
- 54
Beschreibung
Feature
async def func() -> bool:
return True
async def func2() -> None:
if func():
print("hi")
In this code, a likely fix is to change func() to await func(). Meanwhile, it'll always be true since a Coroutine is always true. The truthy-bool check could flag it but it's not enabled by default. We can add a new truthy-coroutine check (like truthy-function) for the specific case of using a Coroutine in boolean context.
The error message can suggest that an await might've been mistakenly omitted.
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 mit dem Vergleich der vorhandenen Prüfungen für truthy-bool und truthy-function und verfolge anschließend, wie ein Coroutine im booleschen Kontext behandelt wird. Füge für das gezeigte async-Beispiel eine truthy-coroutine-Diagnose mit einer Meldung hinzu, die darauf hinweist, dass await fehlen könnte, und überprüfe das Verhalten der Prüfung anhand gleichwertiger Coroutine- und Nicht-Coroutine-Fälle.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- devtools
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100