-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Add support for schema create or update for Swagger, WSDL and OpenApi #6038
Conversation
@dsgouda the build failure is related to an EventHub test. |
@weshaggard Please take a look at the failures. |
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.
LGTM
@dsgouda I wanted to add an extension to one of the models for better usability. Give me an hour. |
@dsgouda I re-queued the failing test leg but I don't believe it is related to the changes in this PR so if it fails again merge anyway. @jsquire looks like another EventHubs test reliability failure
I added a note about this test in issue #5995, and if we see it keep failing then I'll disable it. |
@dsgouda I have pushed my changes. Feel free to merge it as soon as CI passes or fails with EventHub test failure. |
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.
LGTM
@dsgouda can you merge this. |
…Azure#6038) * Add support for schema create or update for Swagger, WSDL and OpenApi * add wsdl extension
Fix based on REST Spec Azure/azure-rest-api-specs#5824