-
Notifications
You must be signed in to change notification settings - Fork 25
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
chore: AOT support for Idempotency #653
base: develop
Are you sure you want to change the base?
chore: AOT support for Idempotency #653
Conversation
…dempotencySerializationContext.cs and Serializer. Update code to use both serializers
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #653 +/- ##
===========================================
+ Coverage 72.99% 73.05% +0.06%
===========================================
Files 196 198 +2
Lines 8013 8068 +55
Branches 856 862 +6
===========================================
+ Hits 5849 5894 +45
- Misses 1877 1883 +6
- Partials 287 291 +4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Quality Gate passedIssues Measures |
Issue number: #212
Summary
Changes
Support for AOT in Idempotency and JMESPath.
Done:
User experience
For serlializing client types inside Idempotency we now have a new method
WithJsonSerializationContext
.This method receives the client JsonSerializationContext
Checklist
Please leave checklist items unchecked if they do not apply to your change.
Is this a breaking change?
RFC issue number:
Checklist:
Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.