eiffel-community / eiffel-community/eiffel-pythonlib
Remove context on callbacks
- Lenguaje dominante
- Python
- Estrellas
- 8
- Forks
- 12
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
### Description
We always call the callbacks with the event object and the context in which this event was sent.
This is a legacy feature that was used when prototyping the eiffellib and should be removed as it does not serve any really good purpose other than clutter in the callbacks.
Note that we need to start by deprecating it and then remove it.
### Motivation
I have found no need for this feature at all and I don't see any real need for it anywhere else.
If there is a need for something like this, then we should instead add a utility module to eiffellib.
For example a function for fetching a link value from an event (get_link("context")).
### Exemplification
Less unnecessary clutter in callbacks.
### Benefits
Less unnecessary clutter in callbacks.
### Possible Drawbacks
Removing this outright will break a lot of tools that rely on it.
Guía de contribución
Línea de trabajo
No se nombran archivos ni pruebas. Empieza localizando dónde reciben los callbacks de eventos de Eiffel el evento y el contexto; después, sigue el rastro de los consumidores de callbacks afectados y de los patrones existentes de compatibilidad o deprecación. La tarea estará terminada cuando el argumento de contexto heredado esté marcado como deprecated antes de eliminarlo, y la interfaz de callbacks y los consumidores afectados se hayan actualizado de forma coherente.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- api, backend
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100