Skip to content

bpo-35213: Where appropriate, use 'macOS' in idlelib. #10478

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

Merged
merged 3 commits into from
Nov 15, 2018

Conversation

terryjreedy
Copy link
Member

@terryjreedy terryjreedy commented Nov 12, 2018

Copy link
Member

@matrixise matrixise left a comment

Choose a reason for hiding this comment

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

Hi @terryjreedy

I found some occurrences of OS X in Doc/library/idle.rst
and in these files:

Lib/idlelib/NEWS.txt
Lib/idlelib/NEWS2x.txt
Lib/idlelib/config.py
Lib/idlelib/editor.py
Lib/idlelib/help.html
Lib/idlelib/macosx.py
Lib/idlelib/mainmenu.py
Lib/idlelib/pyshell.py
Doc/library/idle.rst

the current Mac keyset is named 'IDLE Classic OSX'.
@terryjreedy
Copy link
Member Author

@matrixise Thank you for the search. I made changes in idle.rst (help.html) and config.py, though I am not completely sure of the latter. I explained non-changes in a post to the issue and asked Ned Deily a few questions.

@terryjreedy terryjreedy merged commit b65413b into python:master Nov 15, 2018
@bedevere-bot
Copy link

@terryjreedy: Please replace # with GH- in the commit message next time. Thanks!

@miss-islington
Copy link
Contributor

Thanks @terryjreedy for the PR 🌮🎉.. I'm working now to backport this PR to: 3.6, 3.7.
🐍🍒⛏🤖

@terryjreedy terryjreedy deleted the macOS2 branch November 15, 2018 18:15
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 15, 2018
(cherry picked from commit b65413b)

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
@bedevere-bot
Copy link

GH-10559 is a backport of this pull request to the 3.7 branch.

@bedevere-bot
Copy link

GH-10560 is a backport of this pull request to the 3.6 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 15, 2018
(cherry picked from commit b65413b)

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
miss-islington added a commit that referenced this pull request Nov 15, 2018
(cherry picked from commit b65413b)

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
miss-islington added a commit that referenced this pull request Nov 15, 2018
(cherry picked from commit b65413b)

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants