vmware/pyvmomi

pyVmomi should never use 'print'

Ouverte

#68 ouverte le 26 juin 2014

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (1 personne assignée)Python (807 forks)batch import
enhancementhelp wantedwishlist

Métriques du dépôt

Stars
 (2 141 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

https://docs.python.org/2/library/warnings.html

pyVmomi is a deep level library and it should avoid making use of logging or printing features directly which are application elements we should leave 100% in control of the end application developer.

To close this issue:

  • remove all use of 'print'
  • introduce use of warnings to replace 'print'
  • introduce verbose logging of SOAP messages

Guide contributeur

Direction de recherche
Trouvez toutes les instructions 'print' dans la base de code et remplacez les par warnings.warn pour les messages de dépréciation et introduisez la journalisation pour la verbosité des messages SOAP. Assurez vous de ne pas utiliser print directement.
Stack technique
python
Domaine
backendapi
Type d'issue
Refactorisation
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 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70