vmware/pyvmomi

pyVmomi should never use 'print'

Open

#68 aberto em 26 de jun. de 2014

Ver no GitHub
 (0 comments) (0 reactions) (1 assignee)Python (807 forks)batch import
enhancementhelp wantedwishlist

Métricas do repositório

Stars
 (2.141 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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

Guia do colaborador

Direção de pesquisa
Encontre todas as ocorrências de 'print' na base de código do pyVmomi usando grep ou ferramentas similares. Substitua as pelo módulo warnings do Python para mensagens de depreciação e introduza o módulo logging do Python para registro detalhado de SOAP. O link da documentação fornecido (https://docs.python.org/2/library/warnings.html) mostra como usar warnings. Certifique se de que não haja saída no console para os usuários da biblioteca.
Pilha de tecnologia
python
Domain
backend
Tipo Issue
Refatoração
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 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
basic Pythonfamiliarity with warnings module
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75