vmware/pyvmomi

pyVmomi should never use 'print'

Open

#68 aperta il 26 giu 2014

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)Python (807 fork)batch import
enhancementhelp wantedwishlist

Metriche repository

Star
 (2141 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Trova tutte le occorrenze di 'print' nella codebase di pyVmomi utilizzando grep o strumenti simili. Sostituiscile con il modulo warnings di Python per i messaggi di deprecazione e introduci il modulo logging di Python per la registrazione dettagliata di SOAP. Il link alla documentazione fornito (https://docs.python.org/2/library/warnings.html) mostra come utilizzare warnings. Assicurati che non rimanga alcun output in console per gli utenti della libreria.
Tech stack
python
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
basic Pythonfamiliarity with warnings module
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75