Skip to content

fix(client): match JsonSchemaValidator heading levels to API#733

Merged
TomerAberbach merged 3 commits intoopenai:nextfrom
sweidac:sweidac-patch-1
May 4, 2026
Merged

fix(client): match JsonSchemaValidator heading levels to API#733
TomerAberbach merged 3 commits intoopenai:nextfrom
sweidac:sweidac-patch-1

Conversation

@sweidac
Copy link
Copy Markdown

@sweidac sweidac commented Apr 30, 2026

Increase max allowed nesting depth to match the API docs #732

Increase max allowed nesting depth to match the API docs
@sweidac sweidac requested a review from a team as a code owner April 30, 2026 18:22
@TomerAberbach TomerAberbach changed the base branch from main to next April 30, 2026 18:43
@TomerAberbach TomerAberbach changed the title Update JsonSchemaValidator.kt fix(client): match JsonSchemaValidator heading levels to API Apr 30, 2026
@TomerAberbach
Copy link
Copy Markdown
Collaborator

Mind fixing the tests? (can ignore the "examples" failure)

@sweidac
Copy link
Copy Markdown
Author

sweidac commented May 4, 2026

Hi @TomerAberbach 👋
I fixed them (at least I tried), but didn't run them locally yet due to restrictions. I'll see if i can get around it. Do I need to do something to start the Job on github again? Or will you do it?

@sweidac sweidac force-pushed the sweidac-patch-1 branch from 51cca59 to 78885d2 Compare May 4, 2026 11:18
@TomerAberbach
Copy link
Copy Markdown
Collaborator

I think need to run ./scripts/format to fix lint

@TomerAberbach TomerAberbach merged commit fc57ef7 into openai:next May 4, 2026
4 of 5 checks passed
@stainless-app stainless-app Bot mentioned this pull request May 4, 2026
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.

2 participants