-
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
[TypeSpec] Remove "./" from tspconfig.yaml #24199
Conversation
- Prefix is unnecessary and clutters config
Swagger Validation Report
|
Hi, @mikeharder 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
|
@mikeharder The ApiView link generation failed, please ensure your github account in Azure/Microsoft is public. |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
@@ -14,7 +14,7 @@ emit: [ | |||
] | |||
options: | |||
"@azure-tools/typespec-autorest": | |||
azure-resource-provider-folder: "./data-plane" | |||
azure-resource-provider-folder: "data-plane" | |||
examples-directory: ./examples |
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.
one more case here to remove ./
@v-xuto , GO isn't enabled for typespec DPG. JS and python failed for same reason and isn't related to this PR change.
|
- Prefix is unnecessary and clutters config
Passing CI: https://dev.azure.com/azure-sdk/public/_build/results?buildId=2805265&view=results