eiffel-community / eiffel-community/eiffel-pythonlib

Remove context on callbacks

Abierto
#20 0 comentarios 0 reacciones 0 asignados Ver en GitHub
enhancement
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

Abrir la 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.