pythonnet / pythonnet/pythonnet

Follow up on AssemblyManager.LookupTypes calls

Offen
#1,320 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
C#
Sterne
5.5k
Forks
780
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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).

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, die beiden Aufrufstellen von AssemblyManager.LookupTypes in classobject.cs und genericutil.cs zu lesen, und prüfe anschließend die zugehörigen Änderungen in #1318 sowie die Implementierung von LookupTypes. Ermittle, welche Informationen übereinstimmende Typen unterscheiden können, und definiere das Auswahlverhalten für beide Aufrufstellen. Die Arbeit ist abgeschlossen, wenn keine der beiden mehr blind auf den ersten Treffer vertraut und das Verhalten durch geeignete Tests abgedeckt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
csharp
Bereich
backend
Issue-Typ
Refactoring
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.