-
Notifications
You must be signed in to change notification settings - Fork 19
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
Fix the dataMoverTopic config param always being required #2572
base: development/8.7
Are you sure you want to change the base?
Conversation
Extension configs are validated independently of each other, so validating the dataMoverTopic based on the supportedLifecycleRules param is not currently possible. Issue: BB-619
Hello kerkesni,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files
... and 4 files with indirect coverage changes
@@ Coverage Diff @@
## development/8.7 #2572 +/- ##
===================================================
+ Coverage 70.31% 70.41% +0.10%
===================================================
Files 194 194
Lines 12997 12997
===================================================
+ Hits 9139 9152 +13
+ Misses 3848 3835 -13
Partials 10 10
Flags with carried forward coverage won't be shown. Click here to find out more. |
Extension configs are validated independently of each other, so validating the dataMoverTopic based on the supportedLifecycleRules param is not currently possible.
Issue: BB-619