-
Notifications
You must be signed in to change notification settings - Fork 764
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
Removes Experimental attribute from ResilienceHandler class #5670
Removes Experimental attribute from ResilienceHandler class #5670
Conversation
Removes experimental attribute from ResilienceHandler class
Adds new experimental and stable APIs to the API json file
src/Libraries/Microsoft.Extensions.Http.Resilience/Microsoft.Extensions.Http.Resilience.json
Show resolved
Hide resolved
Suppress LA0006 warning causing the build pipeline to fail
src/Libraries/Microsoft.Extensions.Http.Resilience/Microsoft.Extensions.Http.Resilience.csproj
Show resolved
Hide resolved
Approving on the assumption the @dotnet/dotnet-extensions-resilience team has done the due diligence for the API review. |
Tagging @RussKie and @joperezr as infra-area owners. The PR touches some infra files. Please, review it. Thank you.
Making the API stable was discussed there #4893 (comment). The API looks solid, I think we are good to make it stable. |
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.
Left a couple of minor comments, but looks good otherwise
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
🎉 Good job! The coverage increased 🎉
Full code coverage report: https://dev.azure.com/dnceng-public/public/_build/results?buildId=885446&view=codecoverage-tab |
The PR removes
Experimental
attribute from ResilienceHandler class.Fixes #5669
Microsoft Reviewers: Open in CodeFlow