Skip to content

default config, semantic fallback api , and debug api changes#1

Merged
wenyangfu merged 27 commits intomainfrom
wenyangfu-default-config
Dec 27, 2022
Merged

default config, semantic fallback api , and debug api changes#1
wenyangfu merged 27 commits intomainfrom
wenyangfu-default-config

Conversation

@wenyangfu
Copy link
Owner

Choose a PR Template

Switch to "Preview" on this description then select one of the choices below.

Click here to open a PR for a Data Plane API.

Click here to open a PR for a Control Plane (ARM) API.

@wenyangfu wenyangfu changed the title default config + semantic fallback api changes default config, semantic fallback api , and debug api changes Nov 18, 2022
Comment on lines +1269 to +1271
"SemanticFieldResult": {
"x-ms-client-name": "Semantic",
"properties": {

Choose a reason for hiding this comment

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

I tried to generate the .NET SDK using this swagger and I'm getting below error-
error | PreCheck/DuplicateSchema | Duplicate Schema named 'Semantic' (6 differences)

image

Copy link
Owner Author

Choose a reason for hiding this comment

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

Thanks for letting me know! Are you following these steps to generate the SDK, or are you using another toolchain? https://github.com/Azure/azure-rest-api-specs/tree/master/specification/search/data-plane/Azure.Search

I'd like to be able to generate a copy of the SDK and validate things locally to speed up my dev workflow for swagger.

Choose a reason for hiding this comment

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

You can run dotnet build /t:GenerateCode in the src directory as mentioned here to generate .NET SDK code.

Here are the prerequisites to generate code and work on the azure-sdk-for-net repo.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants