-
Notifications
You must be signed in to change notification settings - Fork 5.4k
[TSP Migration]--elastic #34324
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
base: main
Are you sure you want to change the base?
[TSP Migration]--elastic #34324
Conversation
Next Steps to MergeNext steps that must be taken to merge this PR:
|
PR validation pipeline restarted successfully. If there is ApiView generated, it will be updated in this comment. |
API Change CheckAPIView identified API level changes in this PR and created the following API reviews
|
/azp run SDK Validation - Go |
Azure Pipelines successfully started running 1 pipeline(s). |
* Get detailed properties of a specific Elastic monitor resource, helping you manage observability and performance. | ||
*/ | ||
#suppress "@azure-tools/typespec-azure-core/no-openapi" "FIXME: Update justification, follow aka.ms/tsp/conversion-fix for details" | ||
@operationId("Monitors_Get") |
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.
@welovej I notice that this interface contains many ops which has different prefix of operationId including Monitors_ / DeploymentInfo_
. Is it possible to group the ops with same opreationId prefix into same interface?
Lack of |
/** | ||
* This is the interface that implements the standard Azure Resource Manager operation that returns | ||
* all supported RP operations. You should have exactly one declaration for each | ||
* Azure Resource Manager service. It implements | ||
* GET "/providers/Microsoft.ContosoProviderHub/operations" | ||
*/ | ||
interface Operations { |
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.
Why not use the standard Azure.ResourceManager.Operations
?
This PR migrates your latest version (identified by the tag in your readme.md) of swagger to TypeSpec. We already tried our best to make sure the TypeSpec represents same as previous swagger. Since we lack the business knowledge, please validate this PR again to make sure it's functional equivalent as before. The local validation step is at Getting started | TypeSpec Azure
Besides, TypeSpec encourages to follow ARM guidelines. Therefore, some representations in your previous swagger will be fixed to follow these guidelines. When you see differences in your local validation, please keep this note in mind.
Please reach out to TypeSpec Discussions Channel if there is any help needed.