Common base class

Ouverte
#1 5 commentaires 2 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
5/5
Temps estimé
Plus d'une semaine
Accessibilité débutants
35/100
Type d'issue
Refactorisation
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
python
Domaine
backend-api-design

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.

Description

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.

Langage dominant
Python
Étoiles
69
Forks
30
Merge moyen
9 h 33 min
PR mergées (30 j)
6

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de robotframework/PythonLibCore

Toutes les issues de robotframework/PythonLibCore

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.