vmware/pyvmomi

pyVmomi should never extend base or core types

Open

#69 ouverte le 26 juin 2014

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Python (807 forks)batch import
bughelp wantedhigh priority

Métriques du dépôt

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

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.

Guide contributeur

Direction de recherche
Le problème pointe vers VmomiSupport.py#L179 où la classe Link étend un type de base (par exemple, dict ou list). L'objectif est de refactoriser la classe Link pour qu'elle n'étende pas un type de base. Étudiez l'utilisation actuelle de Link dans la base de code et envisagez d'utiliser la composition ou une classe wrapper. Examinez comment l'extension du type de base est utilisée ailleurs dans le code pour comprendre l'impact. Il n'y a pas de commentaires ni de PR liés, donc la solution est ouverte.
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
understanding of Python metaclassesfamiliarity with pyvmomi codebase
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
20