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

[Storage] HttpResponse changes (new generated code) #36546

Conversation

vincenttran-msft
Copy link
Member

@vincenttran-msft vincenttran-msft commented Jul 19, 2024

This PR contains the fixes necessary on the Storage side to support the new HttpResponse objects being used in generated code.

TODO:

  • Add back in our "great" retry logic from sync to async [x]
  • Make _shared all consistent [x]

@github-actions github-actions bot added the Storage Storage Service (Queues, Blobs, Files) label Jul 19, 2024
@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

@vincenttran-msft
Copy link
Member Author

/azp run python - storage - tests

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vincenttran-msft
Copy link
Member Author

/azp run python - storage - tests

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vincenttran-msft
Copy link
Member Author

/azp run python - storage - tests

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vincenttran-msft
Copy link
Member Author

/azp run python - storage - tests

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vincenttran-msft
Copy link
Member Author

/check-enforcer override

@vincenttran-msft vincenttran-msft merged commit ca110c9 into Azure:main Aug 22, 2024
45 of 49 checks passed
@vincenttran-msft vincenttran-msft deleted the vincenttran/response_changes_postgen branch August 22, 2024 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants