Skip to content

Fix typo in __match_args__ documentation. #98549

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

Closed
wants to merge 3 commits into from

Conversation

icecream17
Copy link

No description provided.

@bedevere-bot bedevere-bot added awaiting review docs Documentation in the Doc dir skip news labels Oct 22, 2022
@ghost
Copy link

ghost commented Oct 22, 2022

All commit authors signed the Contributor License Agreement.
CLA signed

Copy link
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for the fix! I can merge this as soon as the CLA is signed.

@icecream17
Copy link
Author

unfortunately, I'm using my school's chromebook which blocks the site, so this will take a while...

@AlexWaygood
Copy link
Member

No worries, there's no rush :)

@iritkatriel iritkatriel changed the title Fix grammar in __match_args__ documentation. Fix typo in __match_args__ documentation. Nov 25, 2022
@iritkatriel
Copy link
Member

@icecream17 We can accept a typo fix without CLA, but do sign it so you can make more meaty contributions in the future.

@iritkatriel iritkatriel changed the title Fix typo in __match_args__ documentation. fix typo in __match_args__ documentation. Nov 25, 2022
@iritkatriel iritkatriel changed the title fix typo in __match_args__ documentation. Fix typo in __match_args__ documentation. Nov 25, 2022
@terryjreedy
Copy link
Member

Steven, I made of copy PR, #99785, so we can proceed with this.

AlexWaygood pushed a commit that referenced this pull request Nov 26, 2022
A opy of #98549, whose author (@icecream17) uses a school computer that blocks the CLA site. I did not mention this in commit comment above so CLA bot does not pick up the name and request the CLA again.
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 26, 2022
A opy of pythonGH-98549, whose author (@icecream17) uses a school computer that blocks the CLA site. I did not mention this in commit comment above so CLA bot does not pick up the name and request the CLA again.
(cherry picked from commit a86d854)

Co-authored-by: Terry Jan Reedy <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 26, 2022
A opy of pythonGH-98549, whose author (@icecream17) uses a school computer that blocks the CLA site. I did not mention this in commit comment above so CLA bot does not pick up the name and request the CLA again.
(cherry picked from commit a86d854)

Co-authored-by: Terry Jan Reedy <[email protected]>
miss-islington added a commit that referenced this pull request Nov 26, 2022
A opy of GH-98549, whose author (@icecream17) uses a school computer that blocks the CLA site. I did not mention this in commit comment above so CLA bot does not pick up the name and request the CLA again.
(cherry picked from commit a86d854)

Co-authored-by: Terry Jan Reedy <[email protected]>
miss-islington added a commit that referenced this pull request Nov 26, 2022
A opy of GH-98549, whose author (@icecream17) uses a school computer that blocks the CLA site. I did not mention this in commit comment above so CLA bot does not pick up the name and request the CLA again.
(cherry picked from commit a86d854)

Co-authored-by: Terry Jan Reedy <[email protected]>
@icecream17 icecream17 deleted the patch-2 branch January 17, 2023 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting merge docs Documentation in the Doc dir needs backport to 3.10 only security fixes needs backport to 3.11 only security fixes skip issue skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants