-
Notifications
You must be signed in to change notification settings - Fork 918
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
[OSCI] Removed KUI usage in visualizations -- Completion of Original PR #6360
Conversation
This PR completes opensearch-project#5462 Please refer to the above PR for more details Signed-off-by: Anan Zhuang <ananzh@amazon.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6360 +/- ##
==========================================
- Coverage 67.52% 59.49% -8.03%
==========================================
Files 3378 3050 -328
Lines 65883 58556 -7327
Branches 10659 9206 -1453
==========================================
- Hits 44485 34839 -9646
- Misses 18854 21841 +2987
+ Partials 2544 1876 -668
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@SuZhou-Joe thanks for pointing it out. will submit a PR to fix these tests in 5mins |
@SuZhou-Joe PR is here #6411 |
…PR (#6360) (#6407) This PR completes #5462 Please refer to the above PR for more details Signed-off-by: Anan Zhuang <ananzh@amazon.com> (cherry picked from commit 46b17e4) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Description
This PR completes
#5462
Please refer to the above PR for more details
Issues Resolved
Partical #3386
Screenshot
Check List
yarn test:jest
yarn test:jest_integration