google/python-fire

Logging messages don't using logging

Open

#353 aberto em 26 de ago. de 2021

Ver no GitHub
 (4 comments) (2 reactions) (0 assignees)Python (26.214 stars) (1.418 forks)batch import
enhancementhelp wanted

Description

https://github.com/google/python-fire/blob/c1266d0dbb2114514fcf8be62044344b5a51c733/fire/core.py#L239 https://github.com/google/python-fire/blob/c1266d0dbb2114514fcf8be62044344b5a51c733/fire/core.py#L287

When I get the help message, includes INFO: messages like: INFO: Showing help with the command {cmd}

I would like such logging messages to use Python's logging framework which would allow me to suppress and redirect such messages.

Guia do colaborador

Pilha de tecnologia
python
Domain
clideveloper experience
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
fresh
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
basic knowledge of Python loggingfamiliarity with fire/core.py
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80
Direção de pesquisa
The issue points to specific lines in fire/core.py (lines 239 and 287) where INFO messages are printed directly. Replace these print statements with calls to Python's logging module, e.g., logging.getLogger( name ).info(). Ensure the logging configuration does not interfere with normal CLI output, and consider adding a note in documentation about how to suppress these messages.