[DNM] Validate typespec-azure#2974 #36014
Closed
+37,554
−8,594
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* [Monitor Ingestion] Add typespec files Signed-off-by: Paul Van Eck <[email protected]> * Remove ServiceTraits Signed-off-by: Paul Van Eck <[email protected]> * Adjustments Signed-off-by: Paul Van Eck <[email protected]> * Syntax updates Signed-off-by: Paul Van Eck <[email protected]> * Updates Signed-off-by: Paul Van Eck <[email protected]> * Updating tspconfig.yaml * Updating tspconfig.yaml * test update * Updating tspconfig.yaml * Updating tspconfig.yaml * test update * test updates * test updates * test updates * test updates * test updates * add customization class to tspconfig * remove error in tspconfig * Restoring deleted file * Updating tspconfig * Updating formatting --------- Signed-off-by: Paul Van Eck <[email protected]> Co-authored-by: Paul Van Eck <[email protected]>
* [Monitor Query] Add TypeSpec for Logs Query Signed-off-by: Paul Van Eck <[email protected]> * Create custom client Only generate operations we care about. Signed-off-by: Paul Van Eck <[email protected]> * Adjustments Signed-off-by: Paul Van Eck <[email protected]> * Make service version as path param * suppress missing api version warning * Updates Signed-off-by: Paul Van Eck <[email protected]> * Rename examples Signed-off-by: Paul Van Eck <[email protected]> * Recompile autorest with examples Signed-off-by: Paul Van Eck <[email protected]> * Add API version docstring Signed-off-by: Paul Van Eck <[email protected]> * Misc updates Signed-off-by: Paul Van Eck <[email protected]> * Move generated swagger Signed-off-by: Paul Van Eck <[email protected]> * Add readme to pass avocado check Signed-off-by: Paul Van Eck <[email protected]> * Change rows type Signed-off-by: Paul Van Eck <[email protected]> * Add autorest suppressions Signed-off-by: Paul Van Eck <[email protected]> * Rename directory Signed-off-by: Paul Van Eck <[email protected]> --------- Signed-off-by: Paul Van Eck <[email protected]> Co-authored-by: Srikanta <[email protected]>
Signed-off-by: Paul Van Eck <[email protected]>
Signed-off-by: Paul Van Eck <[email protected]>
* Copy files from stable/2025-03-01 Copied the files in a separate commit. This allows reviewers to easily diff subsequent changes against the previous spec. * Update version to stable/2025-07-15 Updated the API version from stable/2025-03-01 to stable/2025-07-15. * Added tag for 2025-07-15 in readme file * creates a new stable version for the enforcement endpoints * Adds the enforcement examples * adds GroupQuotaUsages folder * Adds enforcement definitions * Modifies the api version in the enforcement examples * adds proper api version for usages example * changes the retry after type and fixes typos * changes another description * renames GroupQuotasEnforcementResponse to GroupQuotaEnforcementStatus * formats object * Changes GroupQuotaEnforcementResponse to GroupQuotaEnforcementStatus. reverts unneccessary change in 2023-06-01 preview * ran prettier on 2023-06-01-preview for prettier check violation * changes object definition in preview version as well * resolves retry after issue * reverts the preview file objects
* [Monitor Query Logs] Add Java customizations * format
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. |
…al (Azure#36017) * remove properties order fix and fix location to be optional * remove needless suppression
* add client customizations for csharp * some more updates
Co-authored-by: Zunli Hu <[email protected]>
* fix type * format
Co-authored-by: Will Huang <[email protected]>
pshao25
reviewed
Jul 22, 2025
| @@ -36,5 +36,6 @@ interface Apis { | |||
| @doc("Deletes specified API.") | |||
| delete is ArmResourceDeleteSync<Api>; | |||
| @doc("Returns a collection of APIs.") | |||
| @list | |||
There was a problem hiding this comment.
I assume we don't need this since we should have this @list since we are using the standard template.
There was a problem hiding this comment.
I think so but the template also lets you define a custom response which in theory could be not pageable so need to double check that is not being misued.
|
PR validation pipeline can not start as the pull request is not merged or mergeable - most likely it has merge conflicts. |
…pi-specs into validate-pr-2974
github-actions
bot
added
the
brownfield
|
restarted in a new pr #36153 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.
Click here to open a PR for only SDK configuration.