-
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 plugins owned by @elastic/appex-sharedux
#204959
base: main
Are you sure you want to change the base?
Sustainable Kibana Architecture: Move plugins owned by @elastic/appex-sharedux
#204959
Conversation
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here. |
@elastic/appex-sharedux
Pinging @elastic/appex-sharedux (Team:SharedUX) |
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.
appex-qa
changes 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.
🚀
@elasticmachine merge upstream |
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.
DW changes are only related to a single URL in a comment.
@@ -164,8 +164,8 @@ const getPipeline = (filename: string, removeSteps = true) => { | |||
if ( | |||
(await doAnyChangesMatch([ | |||
/^x-pack\/platform\/packages\/shared\/ai-infra/, | |||
/^x-pack\/platform\/plugins\/shared\/ai_infra/, | |||
/^x-pack\/platform\/plugins\/shared\/inference/, | |||
/^x-pack\/platform\/platform\/plugins\/shared\/shared\/ai_infra/, |
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.
These paths don't look right, the "platform" part seems duplicated. (More occurrences in the file)
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.
Thank you! adjusted manually
cc @gsoldevila
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.
Paths testing in pipeline.ts doesn't look right
…sant/kibana into kbn-team-1309-relocate-20241219135219
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.
single file impact, change LGTM for the Threat Hunting Investigations team
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.
codeowner changes lgtm
…elocate-20241219135219
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.
From EUI team side, looks good 👍🏻
…elocate-20241219135219 # Conflicts: # .buildkite/scripts/pipelines/pull_request/pipeline.ts # .eslintignore # .github/CODEOWNERS # .i18nrc.json # docs/developer/plugin-list.asciidoc # package.json # src/dev/storybook/aliases.ts # src/plugins/telemetry/schema/oss_platform.json # tsconfig.base.json # x-pack/.i18nrc.json # x-pack/platform/plugins/private/canvas/storybook/canvas_webpack.ts # x-pack/plugins/telemetry_collection_xpack/schema/xpack_platform.json # x-pack/plugins/telemetry_collection_xpack/schema/xpack_plugins.json
💛 Build succeeded, but was flaky
Failed CI StepsMetrics [docs]Async chunks
Page load bundle
History
|
Summary
Part of #203163
Relocating only shared-ux-owned plugins for now . Tip on how to review changes if GitHub UI is unmanageable - https://elastic.slack.com/archives/C07TCKTA22E/p1734606026689389