-
Notifications
You must be signed in to change notification settings - Fork 51
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.
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
typespec-autorest - @Openapi.extension
emit schema for Types
#2306
base: main
Are you sure you want to change the base?
Conversation
@Openapi.extension
emit schema for Types
All changed packages have been documented.
Show changes
|
You can try these changes here
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am a tiny bit worried about this change for the value it brings. Wondering if we might be better to just cut of the type conversion for 1.0 so at least using a type is exposed as broken with teh codefix to the user and maybe we add the feature parity in a few month.
Here I am worried that a lot of 3p customers might not pulled the last version and this would be a bigger break(quite confusing I assume as well) with no codefix
Depends on microsoft/typespec#6229
Fixes microsoft/typespec#6077
This PR updates
@azure-tools/typespec-autorest
to support emitting schemas for Types passed into the@OpenAPI.extension
decorator.