-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Rename ResilienceStrategy to ResiliencePipeline #1483
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1483 +/- ##
=======================================
Coverage 83.89% 83.89%
=======================================
Files 278 278
Lines 6496 6496
Branches 1016 1016
=======================================
Hits 5450 5450
Misses 837 837
Partials 209 209
Flags with carried forward coverage won't be shown. Click here to find out more.
|
@martintmk Did I miss something? I though you are against the usage of the term pipeline? |
We did some brainstorming and realized we need to differentiate between "pipeline" (was strategy before) and a component of a pipeline (individual resilience strategy). The end consumer (majority) of an API only cares about The resilience author (minority) cares about |
Details on the issue fix or feature implementation
Based on the API review:
#1233 (comment)
Confirm the following