Skip to content
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

[Backport 2.x] [Workspace] Feature/workspace service core module #5416

Merged
merged 3 commits into from
Dec 14, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 9e3e3a7 from #5092.

* feat: add core workspace module (#145)

The core workspace module(WorkspaceService) is a foundational component
that enables the implementation of workspace features within OSD
plugins. The purpose of the core workspace module is to provide
a framework for workspace implementations.

This module does not implement specific workspace
functionality but provides the essential infrastructure for plugins to
extend and customize workspace features, it maintains a shared
workspace state(observables) across the entire application to ensure
a consistent and up-to-date view of workspace-related information to
all parts of the application.

Signed-off-by: Yulong Ruan <ruanyl@amazon.com>
Signed-off-by: Yulong Ruan <ruanyu1@gmail.com>
Co-authored-by: Miki <amoo_miki@yahoo.com>
(cherry picked from commit 9e3e3a7)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

# Conflicts:
#	CHANGELOG.md
Copy link

codecov bot commented Nov 1, 2023

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (b14f5e8) 67.01% compared to head (3e59b67) 67.01%.
Report is 1 commits behind head on 2.x.

Files Patch % Lines
src/core/public/workspace/workspaces_service.ts 88.88% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              2.x    #5416      +/-   ##
==========================================
- Coverage   67.01%   67.01%   -0.01%     
==========================================
  Files        3289     3291       +2     
  Lines       63304    63338      +34     
  Branches    10057    10061       +4     
==========================================
+ Hits        42426    42446      +20     
- Misses      18439    18452      +13     
- Partials     2439     2440       +1     
Flag Coverage Δ
Linux_1 35.23% <26.66%> (-0.01%) ⬇️
Linux_2 55.23% <94.11%> (+0.05%) ⬆️
Linux_3 43.77% <26.66%> (-0.03%) ⬇️
Linux_4 35.46% <26.66%> (-0.01%) ⬇️
Windows_1 35.26% <26.66%> (-0.04%) ⬇️
Windows_2 55.19% <94.11%> (+0.05%) ⬆️
Windows_3 43.79% <26.66%> (-0.02%) ⬇️
Windows_4 35.46% <26.66%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@joshuarrrr
Copy link
Member

Whitesource failure can be ignored.

ananzh
ananzh previously approved these changes Dec 12, 2023
SuZhou-Joe
SuZhou-Joe previously approved these changes Dec 13, 2023
Signed-off-by: Anan Zhuang <ananzh@amazon.com>
@ananzh ananzh dismissed stale reviews from SuZhou-Joe and themself via 2ed434a December 13, 2023 17:48
Fix lint

Signed-off-by: Miki <miki@amazon.com>

Signed-off-by: Miki <miki@amazon.com>
@ananzh ananzh merged commit 60247dd into 2.x Dec 14, 2023
65 of 66 checks passed
@github-actions github-actions bot deleted the backport/backport-5092-to-2.x branch December 14, 2023 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autocut Skip the changelog verification check on backports first-time-contributor v2.12.0 workspace
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants