-
-
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
[DO NOT MERGE] To verfiy build for Simmy docs PR #1894
Conversation
# Conflicts: # src/Polly.Core/PublicAPI.Unshipped.txt
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1894 +/- ##
=======================================
Coverage 84.66% 84.66%
=======================================
Files 309 309
Lines 6836 6836
Branches 1052 1052
=======================================
Hits 5788 5788
Misses 839 839
Partials 209 209
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
||
## Diagrams | ||
|
||
### Happy path sequence diagram |
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.
It might make sense to explain why fault injection did not happen.
Or rename the Happy path
to No fault injected
/ No injection
|
||
## Diagrams | ||
|
||
### Happy path sequence diagram |
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.
It might make sense to explain why fault injection did not happen.
Pushed changes to the original PR |
Pull Request
The issue or feature being addressed
#1883
Details on the issue fix or feature implementation
This PR is just a collection of fixing commits until I can collaborate on @vany0114's branch.
Confirm the following