[Docs] Add missing team_member_budget_duration param to new_team docstring#24243
Merged
yuneng-jiang merged 2 commits intomainfrom Mar 20, 2026
Merged
[Docs] Add missing team_member_budget_duration param to new_team docstring#24243yuneng-jiang merged 2 commits intomainfrom
yuneng-jiang merged 2 commits intomainfrom
Conversation
Fixes CI failure in test_api_docs.py which validates that all Pydantic model fields are documented in endpoint docstrings. Co-Authored-By: Claude Opus 4.6 <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
Greptile SummaryThis PR adds a single missing docstring entry —
Confidence Score: 5/5
|
| Filename | Overview |
|---|---|
| litellm/proxy/management_endpoints/team_endpoints.py | Added missing team_member_budget_duration docstring entry to the new_team endpoint, fixing the test_api_docs.py CI check. Change is minimal, correctly placed, and consistent with surrounding documentation format. |
Sequence Diagram
sequenceDiagram
participant Client
participant new_team endpoint
participant Pydantic Model (NewTeamRequest)
Client->>new_team endpoint: POST /team/new (team_member_budget_duration=...)
new_team endpoint->>Pydantic Model (NewTeamRequest): Validate request body
Pydantic Model (NewTeamRequest)-->>new_team endpoint: Validated data (incl. team_member_budget_duration)
new_team endpoint-->>Client: 200 OK (team created)
note over new_team endpoint: Docstring now documents<br/>team_member_budget_duration<br/>fixing test_api_docs.py CI
Last reviewed commit: "Merge branch 'main' ..."
ryan-crabbe
approved these changes
Mar 20, 2026
Contributor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Failure Path (Before Fix)
The
test_api_docs.pyCI check validates that all Pydantic model fields on management endpoints have corresponding docstring entries. Thenew_teamendpoint was missing documentation for theteam_member_budget_durationparameter, causing CI to fail.Fix
Added the missing
team_member_budget_durationparameter documentation to thenew_teamdocstring inteam_endpoints.py.Testing
test_api_docs.pylocally — passes with no errors.Type
📖 Documentation