vmware/pyvmomi

pyVmomi should never extend base or core types

Open

#69 aberto em 26 de jun. de 2014

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Python (807 forks)batch import
bughelp wantedhigh priority

Métricas do repositório

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

Description

This VmomiSupport.py#L179 is considered extremely bad form in the Python community. Extension of core types ties the code to interpreter internals that may not be intended for exposure. This is akin to using a hidden API and should be avoided or at least isolated so that potential damage is easier to control.

Rewrite the code so that the class Link is not needed anymore or so that Link can operate without directly extending a core type.

Guia do colaborador

Direção de pesquisa
O problema aponta para VmomiSupport.py#L179, onde a classe Link estende um tipo central (por exemplo, dict ou list). O objetivo é refatorar a classe Link para não estender um tipo central. Investigue o uso atual de Link na base de código e considere usar composição ou uma classe wrapper. Observe como a extensão do tipo central é usada em outros lugares do código para entender o impacto. Não há comentários ou PRs vinculados, então a solução é aberta.
Pilha de tecnologia
python
Domain
backendapi
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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
understanding of Python metaclassesfamiliarity with pyvmomi codebase
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
20