GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-139707: Specify `winreg`, `msvcrt` and `winsound` module availability in docs by StanFromIreland · Pull Request #140429 · python/cpython

Conversation

@StanFromIreland

@StanFromIreland StanFromIreland commented Oct 21, 2025 •
edited by github-actions Bot
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@StanFromIreland
Commit
11ffdd1
@bedevere-app bedevere-app Bot added docs Documentation in the Doc dir skip news awaiting review labels Oct 21, 2025
@github-project-automation github-project-automation Bot added this to Docs PRs Oct 21, 2025
@github-project-automation github-project-automation Bot moved this to Todo in Docs PRs Oct 21, 2025
@StanFromIreland StanFromIreland changed the title Specify winreg availability in docs Specify winreg module availability in docs Oct 21, 2025
@StanFromIreland StanFromIreland added the skip issue label Oct 21, 2025
@StanFromIreland

StanFromIreland commented Oct 21, 2025

Copy link
Copy Markdown
Member Author

There are a few more cases, I will add them to this PR.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@StanFromIreland
StanFromIreland marked this pull request as draft October 21, 2025 16:10
@bedevere-app bedevere-app Bot removed the awaiting review label Oct 21, 2025
@StanFromIreland
Add msvcrt and winsound
0f80883
@StanFromIreland
StanFromIreland marked this pull request as ready for review October 21, 2025 16:13
@bedevere-app bedevere-app Bot added the awaiting review label Oct 21, 2025
@StanFromIreland StanFromIreland changed the title Specify winreg module availability in docs Specify winreg, msvcrt and winsound module availability in docs Oct 21, 2025
@StanFromIreland StanFromIreland changed the title Specify winreg, msvcrt and winsound module availability in docs gh-139707: Specify winreg, msvcrt and winsound module availability in docs Oct 21, 2025
@StanFromIreland
StanFromIreland requested review from encukou and zooba October 21, 2025 16:14
aisk
aisk approved these changes Nov 6, 2025
View reviewed changes
@bedevere-app bedevere-app Bot added awaiting core review and removed awaiting review labels Nov 6, 2025
@encukou

encukou commented Nov 7, 2025

Copy link
Copy Markdown
Member

Hmm, I see these all have :platform: Windows in the header. That's a standard Sphinx option but that seems to have no effect since Sphinx 1.1.
Should we remove that or keep it maintained (everywhere)?
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@StanFromIreland

StanFromIreland commented Nov 7, 2025

Copy link
Copy Markdown
Member Author

Should we remove that or keep it maintained (everywhere)?

In my opinion, keeping the existing ones doesn’t have any significant impact, on the other hand, removing them is the same. I don't see any reason to add more. As such, I'm 0 on changing anything with them, although I don't mind removing them all (for consistency, it makes little sense to remove just the three) in this PR either, if you so wish.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@encukou

encukou commented Nov 10, 2025

Copy link
Copy Markdown
Member

I guess that if we do remove them, it should be its own PR.
StanFromIreland reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@encukou
encukou merged commit 13fa313 into python:main Nov 10, 2025
8 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@bedevere-app bedevere-app Bot removed the awaiting core review label Nov 10, 2025
@github-project-automation github-project-automation Bot moved this from Todo to Done in Docs PRs Nov 10, 2025
@encukou encukou added needs backport to 3.13 only security fixes needs backport to 3.14 bugs and security fixes labels Nov 10, 2025
@miss-islington-app

miss-islington-app Bot commented Nov 10, 2025

Copy link
Copy Markdown

Thanks @StanFromIreland for the PR, and @encukou for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@miss-islington-app

miss-islington-app Bot commented Nov 10, 2025

Copy link
Copy Markdown

Thanks @StanFromIreland for the PR, and @encukou for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 10, 2025
@StanFromIreland @miss-islington
pythongh-139707: Specify winreg, msvcrt and winsound module ava…
672471e 
…ilability in docs (pythonGH-140429)

(cherry picked from commit 13fa313)

Co-authored-by: Stan Ulbrych <[email protected]>
@bedevere-app

bedevere-app Bot commented Nov 10, 2025

Copy link
Copy Markdown

GH-141350 is a backport of this pull request to the 3.13 branch.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@bedevere-app bedevere-app Bot removed the needs backport to 3.13 only security fixes label Nov 10, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 10, 2025
@StanFromIreland @miss-islington
pythongh-139707: Specify winreg, msvcrt and winsound module ava…
7cb6cf5 
…ilability in docs (pythonGH-140429)

(cherry picked from commit 13fa313)

Co-authored-by: Stan Ulbrych <[email protected]>
@bedevere-app

bedevere-app Bot commented Nov 10, 2025

Copy link
Copy Markdown

GH-141351 is a backport of this pull request to the 3.14 branch.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Nov 10, 2025
encukou pushed a commit that referenced this pull request Nov 10, 2025
@miss-islington @StanFromIreland
[3.14] gh-139707: Specify winreg, msvcrt and winsound module av…
63a3737 
…ailability in docs (GH-140429) (GH-141351)

(cherry picked from commit 13fa313)

Co-authored-by: Stan Ulbrych <[email protected]>
encukou pushed a commit that referenced this pull request Nov 10, 2025
@miss-islington @StanFromIreland
[3.13] gh-139707: Specify winreg, msvcrt and winsound module av…
1f9f8e5 
…ailability in docs (GH-140429) (GH-141350)

(cherry picked from commit 13fa313)

Co-authored-by: Stan Ulbrych <[email protected]>
@encukou encukou mentioned this pull request Nov 11, 2025
docs should mention that compression.zstd and other modules are optional #139707
Closed
StanFromIreland added a commit to StanFromIreland/cpython that referenced this pull request Dec 6, 2025
@StanFromIreland
pythongh-139707: Specify winreg, msvcrt and winsound module ava…
5a5b0c6 
…ilability in docs (pythonGH-140429)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aisk aisk aisk approved these changes
@encukou encukou Awaiting requested review from encukou
@zooba zooba Awaiting requested review from zooba

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip issue skip news

Projects

Docs PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@StanFromIreland @encukou @aisk
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.