Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[openai] Spring cleanup #33401

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

[openai] Spring cleanup #33401

wants to merge 1 commit into from
+188 −149

Conversation

deyaaeldeen
Copy link
Member

Some quality-of-life edits and improvements:

  • Added deployments.json to .gitignore to prevent accidental commits of deployment files
  • Updated the apiVersion in the Azure OpenAI client to 2025-01-01-preview
  • Renamed environment variables to follow a consistent convention
  • Removed dependency on core by inlining type definitions of error objects
  • Fixed broken updateSnippets.mjs script
  • Deleted obsolete files
  • Fixed a few lint errors in cookbook samples
  • Updated the migration guide to remove the now obsolete limitations in the audio operations

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

@bterlson bterlson Awaiting requested review from bterlson bterlson will be requested when the pull request is marked ready for review bterlson is a code owner

@glharper glharper Awaiting requested review from glharper glharper will be requested when the pull request is marked ready for review glharper is a code owner

@minhanh-phan minhanh-phan Awaiting requested review from minhanh-phan minhanh-phan will be requested when the pull request is marked ready for review minhanh-phan is a code owner

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
OpenAI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@deyaaeldeen