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

[Bug] more "///" are generated by autorest.powershell #1425

Open
YanaXu opened this issue Jan 8, 2025 · 1 comment
Open

[Bug] more "///" are generated by autorest.powershell #1425

YanaXu opened this issue Jan 8, 2025 · 1 comment
Assignees
Labels
bug P2 SDK Related to .NET SDK generation

Comments

@YanaXu
Copy link
Contributor

YanaXu commented Jan 8, 2025

In this Azure/azure-powershell PR and file src/CognitiveServices/CognitiveServices.Management.Sdk/Generated/ICognitiveServicesManagementClient.cs, additional "///" are generated.

Image

@dolauli
Copy link
Contributor

dolauli commented Jan 8, 2025

Hi @VeryEarly, I believe the root cause of the issue is that some comments were not updated when implementing the feature to break long comments into multiple lines. Here are some examples:
https://github.com/Azure/autorest.powershell/blob/main/powershell/plugins/sdk-tweak-model.ts#L442
https://github.com/Azure/autorest.powershell/blob/main/powershell/plugins/sdk-tweak-model.ts#L451

@isra-fel isra-fel added bug P2 SDK Related to .NET SDK generation labels Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug P2 SDK Related to .NET SDK generation
Projects
None yet
Development

No branches or pull requests

4 participants