eiffel-community / eiffel-community/eiffel-pythonlib

Remove context on callbacks

Open
#20 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
8
Forks
12
PR merge metrics
No merged PRs in 30d

Description

### 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.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.