Skip to content

Merge Polly v5.9.0 to master#435

Merged
joelhulen merged 19 commits intomasterfrom
v590
Apr 9, 2018
Merged

Merge Polly v5.9.0 to master#435
joelhulen merged 19 commits intomasterfrom
v590

Conversation

@reisenberger
Copy link
Copy Markdown
Member

  • Allow Timeout.InfiniteTimeSpan (no timeout) for TimeoutPolicy.
  • Add .AsPolicy<TResult>() and .AsAsyncPolicy<TResult>() methods for converting non-generic policies to generic policies.
  • Per Semver, indicates deprecation of overloads and properties intended to be removed or renamed in Polly v6.0

+ Establish PolicyBase for functionality common to Policy and
Policy<TResult>
+ Esablish separate partial class files for the internals of Policy
operation, from the public Execute (etc) overloads
Rationalise the execute API by flagging overloads for removal, with
ObsoleteAttribute.  These overloads have not been part of execution
interfaces, since Polly v5.2.0.
Flag the deprecation of Context.ExecutionGuid.  Rename as
Context.CorrelationId.
Rename Context.ExecutionKey as OperationKey
Fix specs not to use deprecated overloads
Remove undocumented API for custom policies.  Polly v6.0 intends to
provide a more encapsulated way of doing this.
+ #424 (fixes #419) Allow `Timeout.InfiniteTimeSpan` for TimeoutPolicy
+ #425 Rationalise Execute()/ExecuteAsync() overloads: pre-advise ([per SemVer](https://semver.org/#how-should-i-handle-deprecating-functionality)) expected breaking changes for v6.0
+ #426 Better name ExecutionKey and ExecutionGuid: pre-advise ([per SemVer](https://semver.org/#how-should-i-handle-deprecating-functionality)) expected breaking changes for v6.0
+ Add `Context()` public noargs ctor
+ Minor corrections to method visibility
Add .AsPolicy&lt;TResult&gt; and .AsAsyncPolicy&lt;TResult&gt; methods for converting non-generic policies to generic policies.
@joelhulen
Copy link
Copy Markdown
Member

Another fine release :)

@reisenberger reisenberger deleted the v590 branch May 26, 2018 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants