Skip to content

gh-62480: Remove first-person usage for some docs#102295

Closed
nw0 wants to merge 1 commit into
python:mainfrom
nw0:bpo-18280
Closed

gh-62480: Remove first-person usage for some docs#102295
nw0 wants to merge 1 commit into
python:mainfrom
nw0:bpo-18280

Conversation

@nw0

@nw0 nw0 commented Feb 27, 2023

Copy link
Copy Markdown
Contributor

Additionally: clean up the text explanation for one mock example which this commit touches.

In particular I think this addresses the remaining examples in #62480 (comment)

Additionally: clean up the text explanation for one mock example which
this commit touches.
@nw0 nw0 requested review from a team and rhettinger as code owners February 27, 2023 13:28
@bedevere-bot bedevere-bot added docs Documentation in the Doc dir skip news labels Feb 27, 2023
@rhettinger rhettinger removed their request for review March 2, 2023 03:10

@hauntsaninja hauntsaninja left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not sure this is an improvement. It also introduces a typo

@RafaelWO

Copy link
Copy Markdown
Contributor

Hey @nw0, do you mind if I pick this up? 🙂

Except if @hauntsaninja thinks it should be closed?

@nw0

nw0 commented Jul 19, 2025

Copy link
Copy Markdown
Contributor Author

Hey @nw0, do you mind if I pick this up? 🙂

Please do. I'm going to close this PR, as it's not had meaningful feedback.

@nw0 nw0 closed this Jul 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting changes docs Documentation in the Doc dir skip news

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

7 participants