robotframework / robotframework/PythonLibCore
Common base class
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 69
- Forks
- 30
- Merge medio
- 9 h 33 min
- PR fusionados (30 d)
- 6
Descripción
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.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza localizando las definiciones y los usos de HybridCore, DynamicCore y StaticCore. Revisa cómo afectaría una LibraryCore o RobotLibraryCore compartida a la herencia y a las comprobaciones genéricas con isinstance; se considera terminado cuando los cores concretos compartan la nueva clase base sin depender de HybridCore como tipo común.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- backend-api-design
- Tipo de issue
- Refactorización
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100