ASYNC103 for sync functions
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 62/100
Línea de trabajo
Empieza reproduciendo ASYNC103 contra la llamada síncrona mostrada en src/trio/_core/_instrumentation.py e inspecciona cómo la regla distingue entre funciones síncronas y asíncronas. Se considera terminado cuando ya no se notifica el falso positivo, pero los casos previstos de ASYNC103 siguen notificándose; añade o actualiza la prueba de regresión pertinente una vez identificado su lugar.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I don't think sync functions can raise trio.Cancelled, but this is causing ASYNC103:
def call(
self,
hookname: str,
*args: object,
) -> None:
# snip
for instrument in list(self.data[hookname]):
try:
getattr(instrument, hookname)(*args)
except BaseException: # E
self.remove_instrument(instrument)
INSTRUMENT_LOGGER.exception(
"Exception raised when calling %r on instrument %r. "
"Instrument has been disabled.",
hookname,
instrument,
)
Error message:
src/trio/_core/_instrumentation.py:110:20: ASYNC103 BaseException block with a code path that doesn't re-raise the error. Consider adding an `except trio.Cancelled: raise` before this exception handler.
- Lenguaje dominante
- Python
- Estrellas
- 26
- Forks
- 10
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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.
Más de python-trio/flake8-async
-
Rename ASYNC126 to ASYNC4xxAbiertorule improvement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
python-trio/flake8-async#457 · 1 reacción ·
-
Extend ASYNC401 to other formsPosiblemente ocupada @anshurajbisoyi98-ctrl la tomó hace 3 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
python-trio/flake8-async#475 · 1 comentario · 1 reacción ·
-
Warn about unused `noqa`sAbiertonew rule
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
python-trio/flake8-async#454 · 1 comentario ·
-
New rule: `ASYNC233` for blocking `pathlib` methodsPosiblemente ocupada @paranoa233 la tomó hace 91 días. Abiertogood first issue
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
python-trio/flake8-async#396 · 1 comentario ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
python-trio/flake8-async#395 · 8 comentarios ·
Todos los issues de python-trio/flake8-async
Issues similares
-
bug needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
debpalash/VoiceStudio#2624 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Make Catch2 optional when `RDK_BUILD_CPP_TESTS=OFF`Posiblemente ocupada @pechersky la tomó hoy. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 2 días
-
There are a few redundant calls to `fdesc._setCloseOnExec()`Posiblemente ocupada @gudnimg la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día