Skip to content

Added TypeSpec customization support for Java#26916

Merged
mssfang merged 4 commits intouser/travisw/aoai-2023-12-01-preview-typespecfrom
openai/java/typespec-customization
Dec 1, 2023
Merged

Added TypeSpec customization support for Java#26916
mssfang merged 4 commits intouser/travisw/aoai-2023-12-01-preview-typespecfrom
openai/java/typespec-customization

Conversation

@mssfang
Copy link
Copy Markdown
Contributor

@mssfang mssfang commented Dec 1, 2023

No description provided.

weshaggard and others added 4 commits November 30, 2023 11:22
Having one custom-words.txt for the entire repo has been causing a number of merging issues so we are trying to move away from it and instead have teams use override in the cspell configuration file directly.
* Update readme.python.md

* Update readme.python.md

* Update readme.python.md

* Update readme.python.md

---------

Co-authored-by: Yuchao Yan <yuchaoyan@microsoft.com>
…ame. (#26911)

* Update package print name.

* update.

* update name.
@mssfang mssfang self-assigned this Dec 1, 2023
@mssfang mssfang requested a review from yangyuan as a code owner December 1, 2023 07:26
@openapi-pipeline-app
Copy link
Copy Markdown

openapi-pipeline-app Bot commented Dec 1, 2023

Next Steps to Merge

⌛ Please wait, next steps to merge this PR are being evaluated by automation. ⌛

@openapi-pipeline-app
Copy link
Copy Markdown

openapi-pipeline-app Bot commented Dec 1, 2023

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️LintDiff succeeded [Detail] [Expand]
Validation passes for LintDiff.
compared tags (via openapi-validator v2.1.6) new version base version
default default(a4580cd) default(user/travisw/aoai-2023-12-01-preview-typespec)
️️✔️Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️SwaggerAPIView succeeded [Detail] [Expand]
️️✔️TypeSpecAPIView succeeded [Detail] [Expand]
️️✔️ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️⌛PR Summary pending [Detail]
️⌛Automated merging requirements met pending [Detail]
Posted by Swagger Pipeline | How to fix these errors?

@openapi-pipeline-app
Copy link
Copy Markdown

openapi-pipeline-app Bot commented Dec 1, 2023

Swagger Generation Artifacts

️️✔️ApiDocPreview succeeded [Detail] [Expand]
️️✔️SDK Breaking Change Tracking succeeded [Detail] [Expand]

Breaking Changes Tracking

️🔄 azure-sdk-for-python-track2 inProgress [Detail]
️🔄 azure-sdk-for-java inProgress [Detail]
️🔄 azure-sdk-for-go inProgress [Detail]
️🔄 azure-sdk-for-js inProgress [Detail]
️🔄 azure-resource-manager-schemas inProgress [Detail]
️🔄 azure-powershell inProgress [Detail]
️🔄 azure-sdk-for-python inProgress [Detail]
️🔄 azure-sdk-for-net-track2 inProgress [Detail]
Posted by Swagger Pipeline | How to fix these errors?

@openapi-pipeline-app
Copy link
Copy Markdown

openapi-pipeline-app Bot commented Dec 1, 2023

PR validation pipeline restarted successfully. If there is ApiView generated, it will be updated in this comment.

Copy link
Copy Markdown
Member

@weidongxu-microsoft weidongxu-microsoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved to unblock.

But do take opinion from Alan and Srikanta.

@mssfang
Copy link
Copy Markdown
Contributor Author

mssfang commented Dec 1, 2023

Will change the base branch to merge, since all of the new features we are based on are from that branch

@mssfang mssfang changed the base branch from main to user/travisw/aoai-2023-12-01-preview-typespec December 1, 2023 07:41
@mssfang mssfang requested a review from msyyc as a code owner December 1, 2023 07:41
Copy link
Copy Markdown
Member

@jpalvarezl jpalvarezl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you, Shawn!

@mssfang mssfang merged commit f1af967 into user/travisw/aoai-2023-12-01-preview-typespec Dec 1, 2023
@mssfang mssfang deleted the openai/java/typespec-customization branch December 1, 2023 07:44
lmazuel pushed a commit that referenced this pull request Dec 6, 2023
* baseline: add new label with no changes

* reapplication of changes from PR repository

* incremental: chat inputs, including tools

* incremental, in-progress: chat message rework

* incremental: chat completions updated (undoc'd/unverified)

* incremental: provisional completeness for chat

* Incremental: RAI updates

* RAI, data-plane, tsp compilation fixes

* minor updates after starting .NET integration

* small fix to structured text items for vision

* small but important function fix

* trial and error: OYD apiKey->key

* minor edit

* java emitter config merge

* some PR feedback + tool_calls role fix

* small but important: url is optional in protected_material_code

* add missing layer to image_url

* Add speculative finish_details for gpt-4-vision-preview

* Reorging files for extensions

* Using generics

* Removed generics approach

* Fixed various typos and added docs

* Removed super class parameters

* Removed LRO entries in client.tsp, since they are removed from spec anyway

* WIP

* removed LRO related definitions

* Added @client to hide the LRO op

* Moved visibility modifiers to their own source file

* Annotated Key as removed

* Restored LRO as private methods

* PR feedback

* Hid LROs (#26898)

* format, recompile, backport optional filename for whisper

* PR feedback: naming/docs updates

* Added TypeSpec customization support for Java (#26916)

* Update spell checking instructions (#26874)

Having one custom-words.txt for the entire repo has been causing a number of merging issues so we are trying to move away from it and instead have teams use override in the cspell configuration file directly.

* Update readme.python.md (#26894)

* Update readme.python.md

* Update readme.python.md

* Update readme.python.md

* Update readme.python.md

---------

Co-authored-by: Yuchao Yan <yuchaoyan@microsoft.com>

* [Cognitive Services - Azure AI Content Safety] Update package print name. (#26911)

* Update package print name.

* update.

* update name.

* Added TypeSpec customization support for Java

---------

Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
Co-authored-by: ChenxiJiang333 <119990644+ChenxiJiang333@users.noreply.github.com>
Co-authored-by: Yuchao Yan <yuchaoyan@microsoft.com>
Co-authored-by: jiezhou_microsoft <94604058+JieZhou000@users.noreply.github.com>

* Renamed customization class (#26919)

* Update specification/cognitiveservices/OpenAI.Inference/models/completions/chat_completions.tsp

Co-authored-by: Deyaaeldeen Almahallawi <dealmaha@microsoft.com>

* recompile after accepted/committed feedback fix

* PR feedback: non-optional content_filter_results on prompt model

* very minor change: one file fix via npx prettier

* missing second half of prettier fix: copied file to data-plane

---------

Co-authored-by: Jose Alvarez <jp.alvarezl@gmail.com>
Co-authored-by: Jose Alvarez <jpalvarezl@users.noreply.github.com>
Co-authored-by: Shawn Fang <45607042+mssfang@users.noreply.github.com>
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
Co-authored-by: ChenxiJiang333 <119990644+ChenxiJiang333@users.noreply.github.com>
Co-authored-by: Yuchao Yan <yuchaoyan@microsoft.com>
Co-authored-by: jiezhou_microsoft <94604058+JieZhou000@users.noreply.github.com>
Co-authored-by: Deyaaeldeen Almahallawi <dealmaha@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OpenAI OpenAI service

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants