Skip to content

Insert missing apostrophes in ctypes documentation #115090

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 1 commit into from
Feb 23, 2024

Conversation

irismessage
Copy link
Contributor

@irismessage irismessage commented Feb 6, 2024

Trivial documentation typo fix.


📚 Documentation preview 📚: https://cpython-previews--115090.org.readthedocs.build/

@bedevere-app bedevere-app bot added awaiting review docs Documentation in the Doc dir skip news labels Feb 6, 2024
@irismessage
Copy link
Contributor Author

For line 1455, I'm not completely confident that

a thread-local copy of the systems :data:`errno`
variable;

should have an apostrophe added as I have done, it might be more appropriate to remove the "s" from systems since that's how the phrase "system errno" seems to be used in the rest of the document.

@irismessage
Copy link
Contributor Author

This pull request is a re-make of #115089 because I didn't realise renaming the branch on my fork would break the PR

@willingc
Copy link
Contributor

Thanks @joelsgp.

@willingc willingc merged commit ef6074b into python:main Feb 23, 2024
smontanaro pushed a commit to smontanaro/cpython that referenced this pull request Feb 24, 2024
woodruffw pushed a commit to woodruffw-forks/cpython that referenced this pull request Mar 4, 2024
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
LukasWoodtli pushed a commit to LukasWoodtli/cpython that referenced this pull request Jan 22, 2025
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.

2 participants