Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

gh-44968: Add "Reload from Disk" feature to IDLE#141574

Closed
ashm-dev wants to merge 127 commits into
python:mainpython/cpython:mainfrom
ashm-dev:idleashm-dev/cpython:idleCopy head branch name to clipboard
Closed

gh-44968: Add "Reload from Disk" feature to IDLE#141574
ashm-dev wants to merge 127 commits into
python:mainpython/cpython:mainfrom
ashm-dev:idleashm-dev/cpython:idleCopy head branch name to clipboard

Conversation

@ashm-dev

@ashm-dev ashm-dev commented Nov 14, 2025

Copy link
Copy Markdown
Contributor

Comment thread Lib/idlelib/iomenu.py Outdated

@sharktide sharktide left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I also don’t think we should restore the cursor position since a file’s line numbers could also change when it is edited and it could cause confusion.

ashm-dev and others added 2 commits November 15, 2025 09:47
Co-authored-by: R.C.M <sharktidedev@gmail.com>
Removed cursor position restoration logic during file reload.
@ashm-dev

Copy link
Copy Markdown
Contributor Author

I also don’t think we should restore the cursor position since a file’s line numbers could also change when it is edited and it could cause confusion.

Rewrote it, now it won't save cursor position.

@ashm-dev ashm-dev requested a review from sharktide November 15, 2025 07:24

@picnixz picnixz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please rely more on unittest.mock.patch() as well as mock features directly. In addition add a What's New entry in addition to the NEWS entry.

Comment thread Lib/idlelib/idle_test/test_iomenu.py
Comment thread Lib/idlelib/idle_test/test_iomenu.py Outdated
Comment thread Lib/idlelib/idle_test/test_iomenu.py Outdated
Comment thread Lib/idlelib/idle_test/test_iomenu.py Outdated
Comment thread Lib/idlelib/idle_test/test_iomenu.py Outdated
Comment thread Lib/idlelib/idle_test/test_iomenu.py Outdated
@bedevere-app

bedevere-app Bot commented Nov 15, 2025

Copy link
Copy Markdown

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@ashm-dev ashm-dev requested a review from AA-Turner as a code owner November 15, 2025 22:28
@ashm-dev

Copy link
Copy Markdown
Contributor Author

I have made the requested changes; please review again

@python-cla-bot

python-cla-bot Bot commented Dec 8, 2025

Copy link
Copy Markdown

The following commit authors need to sign the Contributor License Agreement:

CLA not signed

@ashm-dev ashm-dev closed this Dec 8, 2025
@ashm-dev ashm-dev deleted the idle branch December 8, 2025 16:00
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.

Morty Proxy This is a proxified and sanitized view of the page, visit original site.