unregister() argument validation relies on assert
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 42/100
Línea de trabajo
The checks live in the unregister() method, which the issue names but does not locate in a file, so search the repository for it. Confirm that assert is what enforces the two conditions, then run the existing unregister tests to see what currently passes. Done means both error cases raise the proposed exceptions, but the maintainers have not yet agreed to the AssertionError breaking change, so check for a decision before writing code.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
🤖 Written by Claude Opus 5.5 via Claude Code for the pluggy maintainers; I prompted it, it did the work, I read it.
unregister() checks "one of name or plugin" and "plugin is not registered" with assert. Under python -O, pm.unregister(unregistered_plugin) silently returns the plugin, and pm.unregister() returns None.
Proposal: raise TypeError when neither argument is given, and ValueError when the plugin is not registered. Code that currently catches AssertionError would need to change.
Found while working on #759.
- Lenguaje dominante
- Python
- Estrellas
- 1.7k
- Forks
- 172
- Merge medio
- 2 d 18 h
- PR fusionados (30 d)
- 21
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 pytest-dev/pluggy
-
Plugin registry compares plugins by equality, not identityPosiblemente ocupada @00200200 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
pytest-dev/pluggy#760 ·
Los mantenedores suelen responder en 1 día
-
Defaulted positional-only hook parameters are forwarded as keywordsPosiblemente ocupada @adenzhou1350 la tomó hace 1 día. Abierto
Dificultad 4/5 1-2 días Aptitud para principiantes 45/100
pytest-dev/pluggy#774 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
list_plugin_distinfo() keeps unregistered pluginsPosiblemente ocupada @RonnyPfannschmidt la tomó hace 3 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 35/100
pytest-dev/pluggy#763 ·
Los mantenedores suelen responder en 1 día
-
Failures during add_hookspecs / historic replay leave partial statePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 50/100
pytest-dev/pluggy#761 ·
Los mantenedores suelen responder en 1 día
-
[Downstream] devpi broken - trying to register `None`Posiblemente ocupada @bluetech la tomó hace 11 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 58/100
pytest-dev/pluggy#749 · 4 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de pytest-dev/pluggy
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
KalicoCrew/kalico#1012 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
esphome/esphome.io#7657 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
VNG-Realisatie/gemma-zaken#2658 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
comp/gateway comp/plugins duplicate P3 type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
NousResearch/hermes-agent#137019 ·
Los mantenedores suelen responder en 1 día