Should logging.Handler.lock allow None?
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 5.1k
- Forks
- 2.1k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 82
Beschreibung
Since https://github.com/python/cpython/commit/74723e11109a320e628898817ab449b3dad9ee96 (in CPython 3.13 and later), a variety of calls of Handler.acquire() and Handler.release() got converted into with Handler.lock.
As a result, unless you're overriding Handler.handle, Handler.lock cannot be None, which it could be previously.
Note that logging.NullHandler does actually set lock to None in the stdlib, but it does override Handler.handle.
I’m unsure about the policy for typeshed is about instance variables whose valid values depend on whether a particular method is overridden.
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 CPython-Commit 74723e11109a320e628898817ab449b3dad9ee96 und vergleiche Handler.lock, acquire(), release() und handle() mit den typeshed-Deklarationen. Prüfe, wie NullHandlers lock und die überschriebene handle() dargestellt werden, und überprüfe anschließend die typeshed-Richtlinie für Instanzvariablen, deren gültige Werte von Überschreibungen abhängen. Als abgeschlossen gilt die Aufgabe, wenn die Annotation das unterstützte CPython-Verhalten widerspiegelt und die Richtlinienfrage geklärt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 30/100