Skip to content
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

gh-93654: Add module docstring to pathlib #94611

Merged
merged 4 commits into from Jul 7, 2022

Conversation

kj7rrv
Copy link
Contributor

@kj7rrv kj7rrv commented Jul 6, 2022

Issue: gh-93654

Automerge-Triggered-By: GH:brettcannon

@kj7rrv
Copy link
Contributor Author

kj7rrv commented Jul 6, 2022

What is the correct way to mark up the name of a class in ReST?

@brettcannon
Copy link
Member

brettcannon commented Jul 6, 2022

/cc @barneygale

What is the correct way to mark up the name of a class in ReST?

I assume you're referring to your news entry, and in that instance you don't mark up the class name.

@kj7rrv
Copy link
Contributor Author

kj7rrv commented Jul 6, 2022

@brettcannon 2acf6de should fix it.

Lib/pathlib.py Outdated Show resolved Hide resolved
kj7rrv and others added 2 commits Jul 7, 2022
Co-authored-by: Éric <merwok@netwok.org>
@bedevere-bot
Copy link

bedevere-bot commented Jul 7, 2022

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

@merwok merwok changed the title gh-93654: Add docstring to pathlib gh-93654: Add module docstring to pathlib Jul 7, 2022
@merwok
Copy link
Member

merwok commented Jul 7, 2022

but I would like someone else to approve the text 🙂

Copy link
Contributor

@barneygale barneygale left a comment

Looks good to me, nice work.

@brettcannon brettcannon added the 🤖 automerge PR will be merged once it's been approved and all CI passed label Jul 7, 2022
@miss-islington miss-islington merged commit afd6a37 into python:main Jul 7, 2022
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 automerge PR will be merged once it's been approved and all CI passed skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants