Don't set logging level when used as a library
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Start at Server.serve and Server._setup_logging, then compare how logging is configured in cli.py. Keep library use from changing application and dependency loggers, while preserving the CLI's intended logging behavior; verify the affected server and CLI paths manually or with the repository's existing tests.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Server.serve calls Server._setup_logging which explicitly sets the logging level to INFO and hooks up the tornado logger to the output too.
As a user of a library, I should be able to be in control about what gets printed to the screen of my app and what dependencies of mine log to where.
I can fix this by monkey-patching the code with:
server = Server()
server._setup_logging = lambda: None
server.serve(root=".")
but I think that by default the library should not alter the loggers and they should instead be set in cli.py, allowing the CLI experience to be nice but giving library users the flexibility to set things how they wish.
- Lenguaje dominante
- Python
- Estrellas
- 1.1k
- Forks
- 148
- Métricas de merge de PR
- Sin PR fusionados en 30 d
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 lepture/python-livereload
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 70/100
lepture/python-livereload#159 · 3 reacciones ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
lepture/python-livereload#288 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
lepture/python-livereload#283 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
lepture/python-livereload#282 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
lepture/python-livereload#279 ·
Todos los issues de lepture/python-livereload
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
FuRongJun-1999/dsh-memory#65 ·
Los mantenedores suelen responder en 1 día
-
ci needs-ac
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Ikalus1988/MisakaNet#2930 ·
Los mantenedores suelen responder en 1 día
-
`FakeBackendV2.run` fails with `NoiseError` on circuits with delays on qubits where T2 > 2·T1Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Qiskit/qiskit-aer#2466 ·
-
area/cli
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100