8000 Fix typos in documentation by xyb · Pull Request #120338 · python/cpython · GitHub
[go: up one dir, main page]

Skip to content

Fix typos in documentation #120338

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

Merg 8000 ed
merged 1 commit into from
Jun 12, 2024
Merged

Fix typos in documentation #120338

merged 1 commit into from
Jun 12, 2024

Conversation

xyb
Copy link
Contributor
@xyb xyb commented Jun 11, 2024

No description provided.

@xyb xyb requested a review from terryjreedy as a code owner June 11, 2024 06:07
@serhiy-storchaka serhiy-storchaka added docs Documentation in the Doc dir skip news needs backport to 3.12 only security fixes needs backport to 3.13 bugs and security fixes labels Jun 11, 2024
Copy link
Member
@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Member
@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Member
@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

Looks like help.html file is generated from:

Help => IDLE Help: Display help.html with proper formatting.
Doc/library/idle.rst (Sphinx)=> Doc/build/html/library/idle.html
(help.copy_strip)=> Lib/idlelib/help.html

@bedevere-app
Copy link
bedevere-app bot commented Jun 11, 2024

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.

@xyb
Copy link
Contributor Author
xyb commented Jun 12, 2024

@sobolevn I've removed the changes of help.html to leave it to issue #120360. Please take a look, thanks.

@sobolevn sobolevn enabled auto-merge (squash) June 12, 2024 11:56
@sobolevn sobolevn merged commit ce3879b into python:main Jun 12, 2024
32 of 33 checks passed
@miss-islington-app
Copy link

Thanks @xyb for the PR, and @sobolevn for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13.
🐍🍒⛏🤖

@miss-islington-app
Copy link

Sorry, @xyb and @sobolevn, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker ce3879bd45e068f8e2a5a214acd234ca44cad53b 3.13

@miss-islington-app
Copy link

Sorry, @xyb and @sobolevn, I could not cleanly backport this to 3.12 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker ce3879bd45e068f8e2a5a214acd234ca44cad53b 3.12

@sobolevn
Copy link
Member

@xyb would you please create manual backport PRs?

@xyb xyb deleted the fixtypos branch June 13, 2024 03:02
@xyb
Copy link
Contributor Author
xyb commented Jun 13, 2024

@xyb would you please create manual backport PRs?

Sure, I'll be working on it soon.

@bedevere-app
Copy link
bedevere-app bot commented Jun 13, 2024

GH-120438 is a backport of this pull request to the 3.13 branch.

xyb added a commit to xyb/cpython that referenced this pull request Jun 13, 2024
(cherry picked from commit ce3879b)

Co-authored-by: Xie Yanbo <xieyanbo@gmail.com>
@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Jun 13, 2024
@bedevere-app
Copy link
bedevere-app bot commented Jun 13, 2024

GH-120439 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Jun 13, 2024
sobolevn pushed a commit that referenced this pull request Jun 13, 2024
sobolevn pushed a commit that referenced this pull request Jun 13, 2024
@terryjreedy
Copy link
Member

Sorry, my networking was down for a week. Thanks for doing the backports. I am looking at the help.html issue.

mrahtz pushed a commit to mrahtz/cpython that referenced this pull request Jun 30, 2024
noahbkim pushed a commit to hudson-trading/cpython that referenced this pull request Jul 11, 2024
estyxx pushed a commit to estyxx/cpython that referenced this pull request Jul 17, 2024
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 skip issue skip news
Projects
None yet
446A
Development

Successfully merging this pull request may close these issues.

4 participants
0