-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[DSIP-32][Master] Add command fetcher strategy for master fetch command #15900
[DSIP-32][Master] Add command fetcher strategy for master fetch command #15900
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #15900 +/- ##
=========================================
Coverage 39.72% 39.72%
- Complexity 5034 5037 +3
=========================================
Files 1349 1353 +4
Lines 45615 45637 +22
Branches 4891 4892 +1
=========================================
+ Hits 18122 18131 +9
- Misses 25578 25591 +13
Partials 1915 1915 ☔ View full report in Codecov by Sentry. |
0e33af2
to
ca44ca5
Compare
c7b771c
to
2e3170c
Compare
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.
LGTM
b36f0d1
to
4abba0d
Compare
91acdab
to
aeb3308
Compare
aeb3308
to
b572a11
Compare
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.
LGTM
Please improve the pr description. |
Quality Gate failedFailed conditions |
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.
LGTM
Purpose of the pull request
close ##15897
Brief change log
Verify this pull request
Test by UT and E2E