Skip to content

📝 Fix duplicated words in docstrings#15116

Merged
svlandeg merged 2 commits intofastapi:masterfrom
AhsanSheraz:fix/docstring-word-duplications
Mar 16, 2026
Merged

📝 Fix duplicated words in docstrings#15116
svlandeg merged 2 commits intofastapi:masterfrom
AhsanSheraz:fix/docstring-word-duplications

Conversation

@AhsanSheraz
Copy link
Copy Markdown
Contributor

Summary

  • Fix "to to" → "to" in HTTPDigest and OpenIdConnect class docstrings
  • Fix "that that" → "that" in OAuth2PasswordRequestForm and OAuth2PasswordRequestFormStrict class docstrings
  • Fix "be be" → "be" in test data string

Files Changed

  • fastapi/security/http.py — removed duplicated "to" in HTTPDigest docstring
  • fastapi/security/oauth2.py — removed duplicated "that" in two docstrings
  • fastapi/security/open_id_connect_url.py — removed duplicated "to" in OpenIdConnect docstring
  • tests/test_pydanticv2_dataclasses_uuid_stringified_annotations.py — removed duplicated "be" in test data and assertion

Test plan

  • ruff check passes on all changed files
  • ruff format --check passes on all changed files
  • pytest tests/test_pydanticv2_dataclasses_uuid_stringified_annotations.py passes

🤖 Generated with Claude Code

@AhsanSheraz
Copy link
Copy Markdown
Contributor Author

AhsanSheraz commented Mar 14, 2026

Hey Maintainers,
can you please add label & have a look, here?

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Mar 14, 2026

Merging this PR will not alter performance

✅ 20 untouched benchmarks


Comparing AhsanSheraz:fix/docstring-word-duplications (42acf9d) with master (98b269b)1

Open in CodSpeed

Footnotes

  1. No successful run was found on master (eb6851d) during the generation of this report, so 98b269b was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@AhsanSheraz AhsanSheraz force-pushed the fix/docstring-word-duplications branch from 081c963 to 69c7fba Compare March 14, 2026 15:30
Fix "to to" and "that that" word duplications in security module docstrings,
and "be be" in test data string.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@AhsanSheraz AhsanSheraz force-pushed the fix/docstring-word-duplications branch from 69c7fba to faaa946 Compare March 14, 2026 15:31
Copy link
Copy Markdown
Contributor

@roli2py roli2py left a comment

Choose a reason for hiding this comment

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

LGTM

@svlandeg svlandeg added the docs Documentation about how to use FastAPI label Mar 16, 2026
Copy link
Copy Markdown
Member

@svlandeg svlandeg left a comment

Choose a reason for hiding this comment

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

Thanks - these are all genuine typo fixes. LGTM.

@svlandeg svlandeg merged commit 31bbb38 into fastapi:master Mar 16, 2026
30 of 31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation about how to use FastAPI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants