Common base class

Offen

#1 erstellt am 05.9.2017 5 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 5/5
Geschätzter Aufwand: Über eine Woche
Anfängerfreundlichkeit: 35/100

Issue-Typ: Refactoring
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: python
Bereich: backend-api-design

Rechercherichtung

Beginne damit, die Definitionen und Verwendungen von HybridCore, DynamicCore und StaticCore zu finden. Prüfe, wie sich ein gemeinsames LibraryCore oder RobotLibraryCore auf Vererbung und generische isinstance-Prüfungen auswirken würde; abgeschlossen ist die Aufgabe, wenn konkrete Cores die neue Basisklasse gemeinsam verwenden, ohne sich auf HybridCore als gemeinsamen Typ zu stützen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

enhancement priority: medium

Currently both DynamicCore and StaticCore extend HybridCore. This is a bit strange inheritance hierarchy in general, but it's especially stupid to check does a library extend any of these by using isinstance(library, HybridCore). It would be better to have a common base class named LibraryCore or RobotLibraryCore that all concrete lib cores extend. It would allow using isinstance(library, RobotLibraryCore).

Until this common base class is implemented, it's probably best to use isinstance(library, (HybridCore, DynamicCore, StaticCore)) with any generic code. That's both more explicit than just using isinstance(library, HybridCore) and also works if and when other cores don't anymore extend HybridCore.

Vorherrschende Sprache: Python
Sterne: 69
Forks: 30
Ø Merge: 9 Std. 33 Min.
Gemergte PRs (30 T.): 6

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus robotframework/PythonLibCore

Nice to have: Option to keep English original keywords as valid Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

robotframework/PythonLibCore#155 · Erstellt am 04.3.2025

Alle Issues in robotframework/PythonLibCore