Skip to content

gh-151950: Fix Sphinx reference warnings in termios docs - #153885

Merged
asvetlov merged 2 commits into
python:mainfrom
harjothkhara:docs-termios-gh-151950
Oct 28, 2026
Merged

asvetlov merged 2 commits into
python:mainfrom
harjothkhara:docs-termios-gh-151950

Conversation

@harjothkhara

@harjothkhara harjothkhara commented Jul 18, 2026 •

Copy link
Copy Markdown
Contributor

This covers the termios.rst file from gh-151950 (part of the
Sphinx-warnings meta issue gh-151940); other files there are handled by
separate per-file PRs.

Fixes the Sphinx nit-picky reference warnings for
Doc/library/termios.rst and removes the file from
Doc/tools/.nitignore so future regressions are caught by CI.

  • Documents the constants the prose already references with .. data::
    directives, following the existing TCSANOW/TCSADRAIN/TCSAFLUSH
    pattern in the same file: the VMIN/VTIME cc indices, the
    TCIFLUSH/TCOFLUSH/TCIOFLUSH flush selectors, the
    TCOOFF/TCOON/TCIOFF/TCION flow actions, and the
    TIOCGWINSZ/TIOCSWINSZ/TIOCGSIZE/TIOCSSIZE window-size requests.
  • The TIOC* entries note they are not defined on all platforms, since
    tcgetwinsize/tcsetwinsize only require one of each pair.

Verified with a full nit-picky build: the file emits no warnings and
Doc/tools/check-warnings.py --fail-if-regression --fail-if-improved
passes. Documentation-only reference fix, so no NEWS entry (skip news).

Document the VMIN/VTIME, TCIFLUSH/TCOFLUSH/TCIOFLUSH,
TCOOFF/TCOON/TCIOFF/TCION and TIOC* window-size constants with
``.. data::`` directives so the ``:const:`` references resolve, and
drop termios.rst from Doc/tools/.nitignore.
@read-the-docs-community

read-the-docs-community Bot commented Jul 18, 2026 •

Copy link
Copy Markdown

@harjothkhara

harjothkhara commented Sep 3, 2026 •

Copy link
Copy Markdown
Contributor Author

termios.rst part of #151950, docs only and CI is green, could someone take a look?

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@asvetlov
asvetlov merged commit 312222f into python:main Oct 28, 2026
34 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in Docs PRs Oct 28, 2026
@asvetlov

Copy link
Copy Markdown
Contributor

Thanks, @harjothkhara!

@hugovk hugovk added needs backport to 3.14 bugs and security fixes needs backport to 3.15 bugs and security fixes labels Oct 28, 2026
@miss-islington-app

Copy link
Copy Markdown

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

@miss-islington-app

Copy link
Copy Markdown

Thanks @harjothkhara for the PR, and @asvetlov for merging it 🌮🎉.. I'm working now to backport this PR to: 3.15.
🐍🍒⛏🤖

@bedevere-app

bedevere-app Bot commented Oct 28, 2026

Copy link
Copy Markdown

GH-160818 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Oct 28, 2026
@bedevere-app

bedevere-app Bot commented Oct 28, 2026

Copy link
Copy Markdown

GH-160819 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 bugs and security fixes label Oct 28, 2026
hugovk pushed a commit that referenced this pull request Oct 28, 2026
…53885) (#160819)

Co-authored-by: Harjoth Khara <harjoth.khara@gmail.com>
Co-authored-by: Claude Sonnet 5.5 <noreply@anthropic.com>
hugovk pushed a commit that referenced this pull request Oct 28, 2026
…53885) (#160818)

Co-authored-by: Harjoth Khara <harjoth.khara@gmail.com>
Co-authored-by: Claude Sonnet 5.5 <noreply@anthropic.com>
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 news

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants