Should logging.Handler.lock allow None?
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 5.1k
- Forks
- 2.1k
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 82
Descripción
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.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con el commit de CPython 74723e11109a320e628898817ab449b3dad9ee96 y compara Handler.lock, acquire(), release() y handle() con las declaraciones de typeshed. Comprueba cómo se representan el lock y el handle() sobrescrito de NullHandler, y después revisa la política de typeshed para las variables de instancia cuyos valores válidos dependen de sobrescrituras. Se considera terminado cuando la anotación refleja el comportamiento compatible de CPython y la cuestión de la política queda resuelta.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- tooling
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 30/100