Skip to content
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

Replaced disallowed HTML chars and added a missing connection string value #26265

Merged
merged 2 commits into from
Oct 17, 2023

Conversation

HeidiSteen
Copy link
Contributor

Fixes an autogen warning emitted on disallowed html characters.

@HeidiSteen HeidiSteen requested review from markweitzel and MushMal and removed request for a team October 17, 2023 14:53
@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Oct 17, 2023

Next Steps to Merge

✔️ All automated merging requirements have been met! Refer to step 4 in the PR workflow diagram (even if your PR is for data plane, not ARM).

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Oct 17, 2023

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]
There are no breaking changes.
compared swaggers (via Oad v0.10.4)] new version base version
searchindex.json 2023-10-01-Preview(f48bdbb) 2023-10-01-Preview(main)
searchservice.json 2023-10-01-Preview(f48bdbb) 2023-10-01-Preview(main)
searchindex.json 2023-11-01(f48bdbb) 2023-11-01(main)
searchservice.json 2023-11-01(f48bdbb) 2023-11-01(main)
️️✔️Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️CredScan succeeded [Detail] [Expand]
There is no credential detected.
️🔄LintDiff inProgress [Detail]
️️✔️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.
️🔄PoliCheck inProgress [Detail]
️️✔️PrettierCheck succeeded [Detail] [Expand]
Validation passes for PrettierCheck.
️️✔️SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️Lint(RPaaS) succeeded [Detail] [Expand]
Validation passes for Lint(RPaaS).
️️✔️PR Summary succeeded [Detail] [Expand]
Validation passes for Summary.
️️✔️Automated merging requirements met succeeded [Detail] [Expand]
Posted by Swagger Pipeline | How to fix these errors?

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Oct 17, 2023

Swagger Generation Artifacts

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

Breaking Changes Tracking

️️✔️ azure-sdk-for-net-track2 succeeded [Detail] [Expand]
  • ️✔️Succeeded [Logs]Release - Generate from 6036655. SDK Automation 14.0.0
    command	pwsh ./eng/scripts/Automation-Sdk-Init.ps1 ../azure-sdk-for-net_tmp/initInput.json ../azure-sdk-for-net_tmp/initOutput.json
    command	pwsh ./eng/scripts/Invoke-GenerateAndBuildV2.ps1 ../azure-sdk-for-net_tmp/generateInput.json ../azure-sdk-for-net_tmp/generateOutput.json
  • ️✔️Azure.Search.Documents [View full logs]  [Release SDK Changes]
    info	[Changelog]
Posted by Swagger Pipeline | How to fix these errors?

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Oct 17, 2023

Generated ApiView

Language Package Name ApiView Link
.Net Azure.Search.Documents There is no API change compared with the previous version

@HeidiSteen HeidiSteen closed this Oct 17, 2023
@HeidiSteen HeidiSteen reopened this Oct 17, 2023
@tjprescott tjprescott merged commit 6036655 into Azure:main Oct 17, 2023
@openapi-pipeline-app
Copy link

Swagger pipeline restarted successfully, please wait for status update in this comment.

jnlycklama pushed a commit that referenced this pull request Nov 8, 2023
…value (#26265)

* Replaced disallowed HTML chars and added a missing value

* Fixed sev-1 policheck error
zman-ms pushed a commit that referenced this pull request Jan 10, 2024
…value (#26265)

* Replaced disallowed HTML chars and added a missing value

* Fixed sev-1 policheck error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants