Skip to content

Comments

[Queues] Fixes for migration deviations from swagger#40616

Draft
catalinaperalta wants to merge 8 commits intofeature/azure-queue-storage-typespecfrom
queues/breaking-changes-fixes
Draft

[Queues] Fixes for migration deviations from swagger#40616
catalinaperalta wants to merge 8 commits intofeature/azure-queue-storage-typespecfrom
queues/breaking-changes-fixes

Conversation

@catalinaperalta
Copy link
Member

This PR addresses deviations in the typespec definition from the swagger.

@github-actions
Copy link

github-actions bot commented Feb 20, 2026

Next Steps to Merge

Important checks have failed. As of today they are not blocking this PR, but in near future they may.
Addressing the following failures is highly recommended:
  • ⚠️ The check named Swagger BreakingChange has failed. To unblock this PR, follow the process at aka.ms/brch.
If you still want to proceed merging this PR without addressing the above failures, refer to step 4 in the PR workflow diagram.

Comment generated by summarize-checks workflow run.

@microsoft-github-policy-service microsoft-github-policy-service bot added the Storage Storage Service (Queues, Blobs, Files) label Feb 20, 2026

/** Represents an array of signed identifiers */
@Xml.name("SignedIdentifiers")
model ListOfSignedIdentifier {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I dont think architects would like this name, but if it is needed for rust it can be changed with the clientName decorator

@github-actions github-actions bot added data-plane TypeSpec Authored with TypeSpec labels Feb 20, 2026
@github-actions
Copy link

github-actions bot commented Feb 20, 2026

API Change Check

APIView identified API level changes in this PR and created the following API reviews

Language API Review for Package
TypeSpec Storage.Queues
Python azure-storage-queue
JavaScript @azure/storage-queue
TypeSpec Storage.Blob

Comment on lines 461 to 463
@Xml.unwrapped
@Xml.name("SignedIdentifier")
items: Array<SignedIdentifier>;
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was marked as wrapped true in the swagger and also as optional

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data-plane Storage Storage Service (Queues, Blobs, Files) TypeSpec Authored with TypeSpec

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants