Skip to content

Annotate get_ipython#15232

Draft
jenshnielsen wants to merge 2 commits into
ipython:mainfrom
jenshnielsen:annotate_get_ipython
Draft

Annotate get_ipython#15232
jenshnielsen wants to merge 2 commits into
ipython:mainfrom
jenshnielsen:annotate_get_ipython

Conversation

@jenshnielsen
Copy link
Copy Markdown
Contributor

I am using this in my code and it would be convenient if it was annotated.

@jenshnielsen jenshnielsen force-pushed the annotate_get_ipython branch from 382a384 to 50496e3 Compare June 1, 2026 07:40
Comment thread IPython/core/debugger.py
"""

shell: InteractiveShell
shell: InteractiveShell | None
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Allowing this to be None breaks downstream type checking but enforcing this to be not None breaks tests. Happy to look more into this but running out of time right now

@Carreau Carreau marked this pull request as draft June 5, 2026 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant