pythonnet / pythonnet/pythonnet

Follow up on AssemblyManager.LookupTypes calls

オープン
#1,320 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
C#
スター
5.5k
フォーク
780
PR マージ指標
30日以内にマージされた PR はありません

説明

In two places (classobject.cs and genericutil.cs) we call into AssemblyManager.LookupTypes and use just the first matching entry. We should look into these instances and check whether we can do a more informed decision on which type to choose exactly.

The respective code was adjusted slightly in #1318 (before we had an obsolete function LookupType that did the same).

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず、classobject.cs と genericutil.cs にある AssemblyManager.LookupTypes の 2 つの呼び出し箇所を読み、次に #1318 の関連する変更と LookupTypes の実装を確認してください。一致する型を区別できる情報を特定し、両方の呼び出し箇所における選択動作を定義してください。どちらも最初の一致結果に盲目的に依存せず、その動作が適切なテストでカバーされれば作業は完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
csharp
領域
backend
issue の種類
リファクタリング
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。