Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix many py2-style prints in docs and commented code. #14432

Merged
merged 2 commits into from May 13, 2024

Conversation

anntzer
Copy link
Contributor

@anntzer anntzer commented May 6, 2024

The main interest was to transform py2-style print statements that still appeared in some docstrings (e.g. for %history) and would thus show up when looking at the docs either interactively or in the rendered website. I also fixed a number of commented-out print statements at the same time, mostly because it was easier for me to just regexp-change all of them so that I could focus on the others.

The main interest was to transform py2-style print statements that still
appeared in some docstrings (e.g. for %history) and would thus show up
when looking at the docs either interactively or in the rendered
website.  I also fixed a number of commented-out print statements at the
same time, mostly because it was easier for me to just regexp-change all
of them so that I could focus on the others.
Copy link
Member

@Carreau Carreau left a comment

Choose a reason for hiding this comment

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

2 minor things I'll apply directly and merge.

IPython/core/tests/tclass.py Outdated Show resolved Hide resolved
IPython/testing/plugin/ipdoctest.py Outdated Show resolved Hide resolved
@Carreau Carreau merged commit 1066e43 into ipython:main May 13, 2024
16 of 19 checks passed
@Carreau Carreau added this to the 8.25 milestone May 13, 2024
@anntzer anntzer deleted the print branch May 13, 2024 09:37
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.

None yet

2 participants