-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Sustainable Kibana Architecture: Move packages under packages/shared-ux/
#205602
Sustainable Kibana Architecture: Move packages under packages/shared-ux/
#205602
Conversation
…sant/kibana into kbn-team-1309-relocate-20250106110819
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.
Reviewing the migration for shared-ux packages, I found that for certain packages there were still some hanging references to what will be come the former location of these packages when this PR is merged, see below
-
kbn-shared-ux-utility
(packages/kbn-shared-ux-utility)persisting references
- dev_docs/key_concepts/performance/case_studies/case_study_top_level_imports.mdx
-
no-data-types
(packages/shared-ux/page/no_data/types)persisting references
- src/platform/packages/shared/shared-ux/page/kibana_template/impl/README.mdx
-
redirect_app
(packages/shared-ux/link/redirect_app/impl)persisting references
- dev_docs/key_concepts/navigation.mdx
…elocate-20250106110819
@eokoneyo, thank you! I fixed those three places |
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.
src/dev/storybook/aliases.ts
src/platform/packages/private/kbn-ui-shared-deps-src/BUILD.bazel
LGTM
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.
Shared UX changes LGTM, thanks for the fixes!
💔 Build Failed
Failed CI StepsTest Failures
Metrics [docs]
History
|
@elasticmachine merge upstream |
Starting backport for target branches: 8.x |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
…-ux/` (elastic#205602) (cherry picked from commit 175916e) # Conflicts: # .github/CODEOWNERS
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…shared-ux/` (#205602) (#205696) # Backport This will backport the following commits from `main` to `8.x`: - [Sustainable Kibana Architecture: Move packages under `packages/shared-ux/` (#205602)](#205602) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Anton Dosov","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-07T09:46:25Z","message":"Sustainable Kibana Architecture: Move packages under `packages/shared-ux/` (#205602)","sha":"175916ef23a09938833966de51ec8d674be28c06","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:SharedUX","backport:prev-minor","plugin groups"],"number":205602,"url":"https://github.com/elastic/kibana/pull/205602","mergeCommit":{"message":"Sustainable Kibana Architecture: Move packages under `packages/shared-ux/` (#205602)","sha":"175916ef23a09938833966de51ec8d674be28c06"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205602","number":205602,"mergeCommit":{"message":"Sustainable Kibana Architecture: Move packages under `packages/shared-ux/` (#205602)","sha":"175916ef23a09938833966de51ec8d674be28c06"}}]}] BACKPORT--> --------- Co-authored-by: kibanamachine <[email protected]>
## Summary Follow up to elastic#205602 (cherry picked from commit 1388cde)
## Summary Follow up to elastic#205602
## Summary Follow up to elastic#205602
Summary
This PR aims at relocating some of the Kibana modules (plugins and packages) into a new folder structure, according to the Sustainable Kibana Architecture initiative.
Important
packages[\/\\]
andplugins[\/\\]
patterns in the source code (Babel and Eslint config files), and update them appropriately..buildkite/scripts/pipelines/pull_request/pipeline.ts
to ensure that any CI pipeline customizations continue to be correctly applied after the changed path names.ts
and.js
files listed in the sections below, as some of them contain relative paths that have been updated.Note
Are you trying to rebase this PR to solve merge conflicts? Please follow the steps describe here.
50 packages(s) are going to be relocated:
@kbn/shared-ux-avatar-solution
src/platform/packages/shared/shared-ux/avatar/solution
@kbn/shared-ux-button-exit-full-screen
src/platform/packages/shared/shared-ux/button/exit_full_screen
@kbn/shared-ux-button-toolbar
src/platform/packages/shared/shared-ux/button_toolbar
@kbn/shared-ux-card-no-data
src/platform/packages/shared/shared-ux/card/no_data/impl
@kbn/shared-ux-card-no-data-mocks
src/platform/packages/shared/shared-ux/card/no_data/mocks
@kbn/shared-ux-card-no-data-types
src/platform/packages/shared/shared-ux/card/no_data/types
@kbn/shared-ux-chrome-navigation
src/platform/packages/shared/shared-ux/chrome/navigation
@kbn/shared-ux-error-boundary
src/platform/packages/shared/shared-ux/error_boundary
@kbn/shared-ux-file-context
src/platform/packages/shared/shared-ux/file/context
@kbn/shared-ux-file-image
src/platform/packages/shared/shared-ux/file/image/impl
@kbn/shared-ux-file-image-mocks
src/platform/packages/shared/shared-ux/file/image/mocks
@kbn/shared-ux-file-mocks
src/platform/packages/shared/shared-ux/file/mocks
@kbn/shared-ux-file-picker
src/platform/packages/shared/shared-ux/file/file_picker/impl
@kbn/shared-ux-file-types
src/platform/packages/shared/shared-ux/file/types
@kbn/shared-ux-file-upload
src/platform/packages/shared/shared-ux/file/file_upload/impl
@kbn/shared-ux-file-util
src/platform/packages/shared/shared-ux/file/util
@kbn/shared-ux-link-redirect-app
src/platform/packages/shared/shared-ux/link/redirect_app/impl
@kbn/shared-ux-link-redirect-app-mocks
src/platform/packages/shared/shared-ux/link/redirect_app/mocks
@kbn/shared-ux-link-redirect-app-types
src/platform/packages/shared/shared-ux/link/redirect_app/types
@kbn/shared-ux-markdown
src/platform/packages/shared/shared-ux/markdown/impl
@kbn/shared-ux-markdown-mocks
src/platform/packages/shared/shared-ux/markdown/mocks
@kbn/shared-ux-markdown-types
src/platform/packages/shared/shared-ux/markdown/types
@kbn/shared-ux-page-analytics-no-data
src/platform/packages/private/shared-ux/page/analytics_no_data/impl
@kbn/shared-ux-page-analytics-no-data-mocks
src/platform/packages/private/shared-ux/page/analytics_no_data/mocks
@kbn/shared-ux-page-analytics-no-data-types
src/platform/packages/private/shared-ux/page/analytics_no_data/types
@kbn/shared-ux-page-kibana-no-data
src/platform/packages/private/shared-ux/page/kibana_no_data/impl
@kbn/shared-ux-page-kibana-no-data-mocks
src/platform/packages/private/shared-ux/page/kibana_no_data/mocks
@kbn/shared-ux-page-kibana-no-data-types
src/platform/packages/private/shared-ux/page/kibana_no_data/types
@kbn/shared-ux-page-kibana-template
src/platform/packages/shared/shared-ux/page/kibana_template/impl
@kbn/shared-ux-page-kibana-template-mocks
src/platform/packages/shared/shared-ux/page/kibana_template/mocks
@kbn/shared-ux-page-kibana-template-types
src/platform/packages/shared/shared-ux/page/kibana_template/types
@kbn/shared-ux-page-no-data
src/platform/packages/shared/shared-ux/page/no_data/impl
@kbn/shared-ux-page-no-data-config
src/platform/packages/shared/shared-ux/page/no_data_config/impl
@kbn/shared-ux-page-no-data-config-mocks
src/platform/packages/shared/shared-ux/page/no_data_config/mocks
@kbn/shared-ux-page-no-data-config-types
src/platform/packages/shared/shared-ux/page/no_data_config/types
@kbn/shared-ux-page-no-data-mocks
src/platform/packages/shared/shared-ux/page/no_data/mocks
@kbn/shared-ux-page-no-data-types
src/platform/packages/shared/shared-ux/page/no_data/types
@kbn/shared-ux-page-solution-nav
src/platform/packages/shared/shared-ux/page/solution_nav
@kbn/shared-ux-prompt-no-data-views
src/platform/packages/shared/shared-ux/prompt/no_data_views/impl
@kbn/shared-ux-prompt-no-data-views-mocks
src/platform/packages/shared/shared-ux/prompt/no_data_views/mocks
@kbn/shared-ux-prompt-no-data-views-types
src/platform/packages/shared/shared-ux/prompt/no_data_views/types
@kbn/shared-ux-prompt-not-found
src/platform/packages/shared/shared-ux/prompt/not_found
@kbn/shared-ux-router
src/platform/packages/shared/shared-ux/router/impl
@kbn/shared-ux-router-mocks
src/platform/packages/shared/shared-ux/router/mocks
@kbn/shared-ux-router-types
src/platform/packages/shared/shared-ux/router/types
@kbn/shared-ux-storybook-config
src/platform/packages/private/shared-ux/storybook/config
@kbn/shared-ux-storybook-mock
src/platform/packages/shared/shared-ux/storybook/mock
@kbn/shared-ux-tabbed-modal
src/platform/packages/shared/shared-ux/modal/tabbed
@kbn/shared-ux-table-persist
src/platform/packages/shared/shared-ux/table_persist
@kbn/shared-ux-utility
src/platform/packages/shared/kbn-shared-ux-utility
Updated relative paths