-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Removing the Anonymous types in public-preview version #24515
Conversation
Hi, @Mohana-Krishna-Nali Thanks for your PR. I am workflow bot for review process. Here are some small tips. Any feedback about review process or workflow bot, pls contact swagger and tools team. vscswagger@microsoft.com |
Swagger Generation Artifacts
|
Generated ApiView
|
Hi @Mohana-Krishna-Nali, one or multiple breaking change(s) is detected in your PR. Please check out the breaking change(s), and provide business justification in the PR comment and @ PR assignee why you must have these change(s), and how external customer impact can be mitigated. Please ensure to follow breaking change policy to request breaking change review and approval before proceeding swagger PR review. |
I see no readme.typescript.md in this folder https://github.com/Azure/azure-rest-api-specs/tree/dev/v-mnali/Clearing-Anonymous-types/specification/managednetworkfabric/resource-manager but it's referenced https://github.com/Azure/azure-rest-api-specs/tree/dev/v-mnali/Clearing-Anonymous-types/specification/managednetworkfabric/resource-manager#typescript |
@qiaozha Author has resolved the comments, please re-review |
* Removing the Anonymous types in private-preview version * Prettier check fix and spell check fix * resolving prettier check * Added the type-script readme file.
Issues:
Solution: