robotframework / robotframework/PythonLibCore
Common base class
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 69
- Forks
- 30
- Merge moyen
- 9 h 33 min
- PR mergées (30 j)
- 6
Description
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.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par localiser les définitions et les utilisations de HybridCore, DynamicCore et StaticCore. Examinez l’impact qu’aurait un LibraryCore ou RobotLibraryCore partagé sur l’héritage et les vérifications génériques avec isinstance ; le travail est terminé lorsque les cores concrets partagent la nouvelle classe de base sans s’appuyer sur HybridCore comme type commun.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- backend-api-design
- Type d'issue
- Refactorisation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100