vmware/pyvmomi

pyVmomi should never extend base or core types

Open

#69 geöffnet am 26. Juni 2014

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (807 Forks)batch import
bughelp wantedhigh priority

Repository-Metriken

Stars
 (2.141 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Das Problem verweist auf VmomiSupport.py#L179, wo die Link Klasse einen Kerntyp (z. B. dict oder list) erweitert. Das Ziel ist, die Link Klasse so zu refaktorieren, dass sie keinen Kerntyp erweitert. Untersuchen Sie die aktuelle Verwendung von Link in der Codebasis und erwägen Sie die Verwendung von Komposition oder einer Wrapper Klasse. Betrachten Sie, wie die Kerntyp Erweiterung an anderer Stelle im Code verwendet wird, um die Auswirkungen zu verstehen. Es gibt keine Kommentare oder verknüpfte PRs, daher ist die Lösung offen.
Tech Stack
python
Domain
backendapi
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
understanding of Python metaclassesfamiliarity with pyvmomi codebase
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
20