vmware/pyvmomi

pyVmomi should never extend base or core types

Open

#69 opened on 2014年6月26日

GitHub で見る
 (0 comments) (0 reactions) (0 assignees)Python (2,141 stars) (807 forks)batch import
bughelp wantedhigh priority

説明

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.

コントリビューターガイド

技術スタック
python
領域
backendapi
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
understanding of Python metaclassesfamiliarity with pyvmomi codebase
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
20
調査方針
The issue points to VmomiSupport.py#L179 where the Link class extends a core type (e.g., dict or list). The goal is to refactor the Link class to not extend a core type. Investigate the current usage of Link in the codebase and consider using composition or a wrapper class. Look at how the core type extension is used elsewhere in the code to understand the impact. No comments or linked PRs exist, so the solution is open ended.