Skip to content

fix: limit long-title exemption to title nodes#201

Open
w3lld1 wants to merge 1 commit into
PyCQA:mainfrom
w3lld1:fix/allow-long-titles-scope
Open

fix: limit long-title exemption to title nodes#201
w3lld1 wants to merge 1 commit into
PyCQA:mainfrom
w3lld1:fix/allow-long-titles-scope

Conversation

@w3lld1

@w3lld1 w3lld1 commented Jul 10, 2026

Copy link
Copy Markdown

Summary

  • restrict allow-long-titles to actual title and subtitle nodes instead of entire section nodes
  • preserve D001 checks for continuation lines inside section paragraphs
  • add regression coverage based on the reported clang-tidy document shape

Validation

  • tox -e py,lint (32 tests passed; coverage threshold, Ruff, mypy, pylint, and all pre-commit hooks passed)
  • git diff --check

Fixes #195

@w3lld1 w3lld1 requested a review from ssbarnea as a code owner July 10, 2026 15:20
@w3lld1 w3lld1 changed the title Fix long-title exemption leaking into section content fix: limit long-title exemption to title nodes Jul 10, 2026
@w3lld1 w3lld1 force-pushed the fix/allow-long-titles-scope branch from bdf0402 to fa77764 Compare July 10, 2026 15:22
@github-actions github-actions Bot added the fix label Jul 10, 2026
@w3lld1

w3lld1 commented Jul 10, 2026

Copy link
Copy Markdown
Author

CI note: the two failing paths appear to be infrastructure failures before the project checks run:

  • pre-commit.ci: pyupgrade crashes under Python 3.14 with TypeError: cannot use a bytes pattern on a string-like object
  • GitHub Actions prepare: coactions/dynamic-matrix@v4 fails while building its container because cffi cannot find cc

The repository's local tox -e py,lint run passes (32 tests, coverage threshold, Ruff, mypy, pylint, and all pre-commit hooks). Read the Docs and the ack check are green.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

allow-long-titles option incorrectly skipping non-title lines

1 participant