google/python-fire

Logging messages don't using logging

Open

#353 ouverte le 26 août 2021

Voir sur GitHub
 (4 commentaires) (2 réactions) (0 assignés)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.

Guide contributeur

Stack technique
python
Domaine
clideveloper experience
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
basic knowledge of Python loggingfamiliarity with fire/core.py
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80
Direction de recherche
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.