Skip to content

pass cell_meta from do_execute to run_cell#1475

Merged
ianthomas23 merged 6 commits intoipython:mainfrom
erawn:cell_meta
Mar 10, 2026
Merged

pass cell_meta from do_execute to run_cell#1475
ianthomas23 merged 6 commits intoipython:mainfrom
erawn:cell_meta

Conversation

@erawn
Copy link
Contributor

@erawn erawn commented Nov 12, 2025

Other half of ipython/ipython#15071

In order to avoid an if-elif-elif-else sequence I copied the strategy already used for execute_request and made a list of the arguments rather than conditioning different method calls. Happy to switch it back or do something else if others prefer a different style.

@erawn erawn changed the title pass cell_meta from do_execute to run_cell [WIP] pass cell_meta from do_execute to run_cell Nov 12, 2025
@erawn erawn changed the title [WIP] pass cell_meta from do_execute to run_cell pass cell_meta from do_execute to run_cell Dec 10, 2025
@ianthomas23
Copy link
Collaborator

Can you rebase this on main as there were some CI failures that I think I have fixed.

Copy link
Collaborator

@ianthomas23 ianthomas23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @erawn

There are some linting and docs errors but I will merge it now and fix the errors myself afterwards, as you have been waiting quite a while for this to be accepted.

@ianthomas23 ianthomas23 merged commit b287255 into ipython:main Mar 10, 2026
39 of 42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants