Skip to content

[OpenAI] Edit Customization Layer with Main Typespec#28533

Merged
minhanh-phan merged 20 commits into
Azure:mainfrom
minhanh-phan:openai/update-customizations
Mar 19, 2024
Merged

[OpenAI] Edit Customization Layer with Main Typespec#28533
minhanh-phan merged 20 commits into
Azure:mainfrom
minhanh-phan:openai/update-customizations

Conversation

@minhanh-phan
Copy link
Copy Markdown
Member

@minhanh-phan minhanh-phan commented Feb 9, 2024

Packages impacted by this PR

@azure/openai

Issues associated with this PR

Describe the problem that is addressed by this PR

This PR updates AOAI package to the latest customization tool v2 with the new folder structure. "sources" folder is removed and the generated code will be located in "generated" folder. The PR does not use the latest TypeSpec commit; that will be addressed in a later PR to avoid massive changes to the API.

What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?

Are there test cases added in this PR? (If not, why?)

Provide a list of related PRs (if any)

Command used to generate this PR:**(Applicable only to SDK release request PRs)

Checklists

  • Added impacted package name to the issue description
  • Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
  • Added a changelog (if necessary)

@github-actions github-actions Bot added the OpenAI label Feb 9, 2024
@azure-sdk
Copy link
Copy Markdown
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

@azure/openai

Comment thread sdk/openai/openai/src/index.ts
@minhanh-phan minhanh-phan marked this pull request as ready for review March 19, 2024 17:45
@minhanh-phan minhanh-phan merged commit eaf655d into Azure:main Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants