Skip to content

Comments

Example of fixing Storage exception issue with breaking change#41587

Closed
alzimmermsft wants to merge 1 commit intoAzure:mainfrom
alzimmermsft:AzStorage_FixExceptionDeserializationIssueBreaking
Closed

Example of fixing Storage exception issue with breaking change#41587
alzimmermsft wants to merge 1 commit intoAzure:mainfrom
alzimmermsft:AzStorage_FixExceptionDeserializationIssueBreaking

Conversation

@alzimmermsft
Copy link
Member

@alzimmermsft alzimmermsft commented Aug 21, 2024

Description

Another design for fixing the Storage exception deserialization issue with a breaking API change. See #41585 for more details.

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

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

azure-sdk commented Aug 21, 2024 •
edited
Loading

API change check

API changes are not detected in this pull request.

@alzimmermsft alzimmermsft deleted the AzStorage_FixExceptionDeserializationIssueBreaking branch August 22, 2024 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JonathanGiles JonathanGiles Awaiting requested review from JonathanGiles JonathanGiles will be requested when the pull request is marked ready for review JonathanGiles is a code owner

@srnagar srnagar Awaiting requested review from srnagar srnagar will be requested when the pull request is marked ready for review srnagar is a code owner

@rujche rujche Awaiting requested review from rujche rujche will be requested when the pull request is marked ready for review rujche is a code owner

@Netyyyy Netyyyy Awaiting requested review from Netyyyy Netyyyy will be requested when the pull request is marked ready for review Netyyyy is a code owner

@saragluna saragluna Awaiting requested review from saragluna saragluna will be requested when the pull request is marked ready for review saragluna is a code owner

@moarychan moarychan Awaiting requested review from moarychan moarychan will be requested when the pull request is marked ready for review moarychan is a code owner

@browndav-msft browndav-msft Awaiting requested review from browndav-msft browndav-msft will be requested when the pull request is marked ready for review browndav-msft is a code owner

@gunjansingh-msft gunjansingh-msft Awaiting requested review from gunjansingh-msft gunjansingh-msft will be requested when the pull request is marked ready for review gunjansingh-msft is a code owner

@ibrandes ibrandes Awaiting requested review from ibrandes ibrandes will be requested when the pull request is marked ready for review ibrandes is a code owner

@kyleknap kyleknap Awaiting requested review from kyleknap kyleknap will be requested when the pull request is marked ready for review kyleknap is a code owner

@seanmcc-msft seanmcc-msft Awaiting requested review from seanmcc-msft seanmcc-msft will be requested when the pull request is marked ready for review seanmcc-msft is a code owner

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

Assignees

No one assigned

Labels

Storage Storage Service (Queues, Blobs, Files)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@alzimmermsft @azure-sdk