Skip to content

chore(no-release): clean configuration comments - #7

Merged
justgithubaccount merged 1 commit into
mainfrom
codex/config-comment-cleanup
Oct 30, 2026
Merged

justgithubaccount merged 1 commit into
mainfrom
codex/config-comment-cleanup

Conversation

@justgithubaccount

Copy link
Copy Markdown
Owner

Summary

  • remove step-by-step and stale comments from FastAPI source
  • keep concise comments for provider-data bounds, OTLP paths, and trace correlation
  • correct the pool_recycle explanation without changing its value
  • keep executable Python behavior unchanged

Companion cleanup to documentation PR #6. This PR is intentionally separate and has no image release or deployment action.

Validation

  • Python AST comparison against base for every changed source file
  • 22 pytest tests
  • Ruff
  • mypy
  • git diff --check

Safety

No API behavior, model default, dependency, secret, image, release, or live service was changed.

@justgithubaccount
justgithubaccount marked this pull request as ready for review October 30, 2026 07:46
@justgithubaccount
justgithubaccount merged commit 899ba64 into main Oct 30, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant