FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix typos in the Lib directory by cclauss · Pull Request #28775 · python/cpython · GitHub

Repository navigation

Fix typos in the Lib directory - #28775

Merged
gpshead merged 3 commits into
python:mainfrom
cclauss:codespell-Lib
Oct 6, 2021
Merged

gpshead merged 3 commits into
python:mainfrom
cclauss:codespell-Lib

Conversation

cclauss commented Oct 6, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor

Like #28744 but for the Lib directory.

[skip issue] Opening a related issue is pending python/psf-infra-meta#130

codespell --ignore-words-list=aline,alo,ans,asend,atleast,ba,bloc,browseable,cant,crasher,crashers,commen,datas,datea,doesnot,donn,ende,filetest,filetests,fiter,fo,hda,hel,hellow,henc,hist,infinit,inout,interruptable,invokations,ist,ket,lable,larg,leapyear,mye,nams,nd,ned,nin,notmutch,nwe,oder,oll,parm,raison,responsable,seh,simeple,squirl,spawnve,sur,tbe,te,teh,tey,thow,timere,tre,trough,tthe,ue,upto,wile,wont --skip=".decTest,.txt,*.types" Lib

terryjreedy 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I checked everything, not just idlelib, and all the changes look correct. I checked a few with the handy web search bar. One other check will be sufficient.

The requested change is to change 'gui' to 'GUI' in a line with another correction.

If whoever merges this does not backport to both 3.10 and 3.9, please let me know as I want at least the idlelib changes backported and will do so if necessary.

Comment thread Lib/email/errors.py
Comment thread Lib/idlelib/idle_test/mock_tk.py Outdated

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.

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
cclauss requested review from terryjreedy and removed request for a team October 6, 2021 21:16

ericvsmith 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Except for the one small change I suggest, these all look good to me.

Comment thread Lib/test/test_time.py Outdated
cclauss requested a review from ericvsmith October 6, 2021 21:25

ericvsmith 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Looks good to me.

gpshead self-assigned this Oct 6, 2021
gpshead removed the request for review from 1st1 October 6, 2021 22:57

gpshead commented Oct 6, 2021

Copy link
Copy Markdown
Member

There are ~5 of these PRs, see #28745 (comment) for details on how we're handling them. TL;DR - I'm taking the reviews. Backports to 3.9 and 3.10 are being done (manually) to reduce potential for merge conflicts of other bugfix backports.

gpshead merged commit 745c9d9 into python:main Oct 6, 2021

Copy link
Copy Markdown
Contributor

Thanks @cclauss for the PR, and @gpshead for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖

Copy link
Copy Markdown
Contributor

Sorry, @cclauss and @gpshead, I could not cleanly backport this to 3.10 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 745c9d9dfc1ad6fdfdf1d07420c6273ff67fa5be 3.10

Copy link
Copy Markdown
Contributor

Sorry @cclauss and @gpshead, I had trouble checking out the 3.9 backport branch.
Please backport using cherry_picker on command line.
cherry_picker 745c9d9dfc1ad6fdfdf1d07420c6273ff67fa5be 3.9

cclauss deleted the codespell-Lib branch October 7, 2021 06:01
cclauss added a commit to cclauss/cpython that referenced this pull request Oct 7, 2021
Fix typos in the Lib directory as identified by codespell.

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
(cherry picked from commit 745c9d9)

Copy link
Copy Markdown

GH-28803 is a backport of this pull request to the 3.9 branch.

cclauss added a commit to cclauss/cpython that referenced this pull request Oct 7, 2021
Fix typos in the Lib directory as identified by codespell.

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>.
(cherry picked from commit 745c9d9)

Co-authored-by: Christian Clauss <cclauss@me.com>

Copy link
Copy Markdown

GH-28804 is a backport of this pull request to the 3.10 branch.

miss-islington pushed a commit that referenced this pull request Oct 7, 2021
Fix typos in the Lib directory as identified by codespell.

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
(cherry picked from commit 745c9d9)

Automerge-Triggered-By: GH:JulienPalard
terryjreedy pushed a commit that referenced this pull request Oct 7, 2021
Fix typos in the Lib directory as identified by codespell.

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>.
(cherry picked from commit 745c9d9)

Co-authored-by: Christian Clauss <cclauss@me.com>
jaraco pushed a commit to jaraco/cpython that referenced this pull request Feb 17, 2023
Fix typos in the Lib directory as identified by codespell.

Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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

Development

Successfully merging this pull request may close these issues.

8 participants


Back | FazBrowse Home | New Git URL