Skip to content

Typo in minidom documentation #93437

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 2 commits into from
Jun 30, 2022
Merged

Typo in minidom documentation #93437

merged 2 commits into from
Jun 30, 2022

Conversation

brandelune
Copy link
Contributor

No description provided.

@ghost
Copy link

ghost commented Jun 2, 2022

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-bot
Copy link

Most changes to Python require a NEWS entry.

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

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels Jun 2, 2022
@brandelune
Copy link
Contributor Author

Is there a reason why a typo takes 1 month to be fixed?

@AlexWaygood
Copy link
Member

Is there a reason why a typo takes 1 month to be fixed?

This probably just fell off the radar. There are a lot of PRs filed against CPython, and not enough core devs to review them all. Please try to be patient, and remember that everybody helping to maintain CPython is a volunteer giving up their free time.

@brandelune
Copy link
Contributor Author

I totally understand and I'm also a volunteer giving up my free time in other project, so I'm not blaming anybody here. Just asking.

Because this is Python, not some obscure free software project, and this is a 1 word typo.

Is there a triage issue (why are core devs involved in reviewing typos?)

@AlexWaygood
Copy link
Member

I totally understand and I'm also a volunteer giving up my free time in other project, so I'm not blaming anybody here.

I know, and thank you for giving up your free time to file this PR! I'm sorry this has taken a little while.

Is there a triage issue (why are core devs involved in reviewing typos?)

This has already been triaged — Adam's a triager, and has already approved it, and I'm also a triager, and have just pinged a core dev for you. But only core devs have merge privileges (and personally, i think that's as it should be), so every PR does ultimately need to be approved by a core dev, even if it's just a trivial typo fix.

There are broader discussions underway at the moment about how we can do better at triaging issues and PRs. I wrote about them last month: https://pyfound.blogspot.com/2022/05/the-2022-python-language-summit-dealing.html?m=1

@JelleZijlstra JelleZijlstra added needs backport to 3.10 only security fixes needs backport to 3.11 only security fixes labels Jun 30, 2022
@ambv
Copy link
Contributor

ambv commented Jun 30, 2022

Rebased to get rid of the Windows build failures.

@JelleZijlstra JelleZijlstra merged commit 639e351 into python:main Jun 30, 2022
@miss-islington
Copy link
Contributor

Thanks @brandelune for the PR, and @JelleZijlstra for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 30, 2022
(cherry picked from commit 639e351)

Co-authored-by: Jean-Christophe Helary <[email protected]>
@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Jun 30, 2022
@bedevere-bot
Copy link

GH-94458 is a backport of this pull request to the 3.11 branch.

@bedevere-bot
Copy link

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 30, 2022
(cherry picked from commit 639e351)

Co-authored-by: Jean-Christophe Helary <[email protected]>
@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Jun 30, 2022
miss-islington added a commit that referenced this pull request Jun 30, 2022
(cherry picked from commit 639e351)

Co-authored-by: Jean-Christophe Helary <[email protected]>
miss-islington added a commit that referenced this pull request Jun 30, 2022
(cherry picked from commit 639e351)

Co-authored-by: Jean-Christophe Helary <[email protected]>
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.

7 participants