CSSUTILS logger overrides existing config on import
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 92
- Forks
- 30
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
Hi!
On first import of `cssutils`, the `'CSSUTILS'` logger is set up and configured with some default values, as can be seen here:
https://github.com/jaraco/cssutils/blob/main/cssutils/errorhandler.py#L51-L56
This can lead to issues if your import of `cssutils` happens later than your main logging config setup, since it will override whatever is already configured. To avoid it, you must pre-emptively import `cssutils` before your own logging config.
It would be easier if `cssutils` detected if the logger has already been set up, and didn't add it's own default config in that case. At the very least, if it preserved the loglevel the user has selected that would help.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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
Lee las líneas 51-56 de cssutils/errorhandler.py y sigue cómo se configura el logger CSSUTILS en la primera importación. Define done como conservar una configuración existente del logger, incluido el nivel de log del usuario, en lugar de aplicar valores predeterminados; el issue no nombra ningún archivo de pruebas, así que identifica la cobertura existente relevante antes de hacer cambios.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- developer-experience
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 42/100