Skip to content

[CI] remove local dependency description from kbn-yaml-loader#263437

Merged
delanni merged 2 commits intoelastic:mainfrom
delanni:remove-kbn-yaml-loader-local-dep
Apr 15, 2026
Merged

[CI] remove local dependency description from kbn-yaml-loader#263437
delanni merged 2 commits intoelastic:mainfrom
delanni:remove-kbn-yaml-loader-local-dep

Conversation

@delanni
Copy link
Copy Markdown
Member

@delanni delanni commented Apr 15, 2026

Summary

Clean installs will fail with

error Error: ENOENT: no such file or directory, lstat '/Users/tkajtoch/dev/kibana/src/platform/packages/shared/kbn-yaml-loader/node_modules/yaml

It's probably because we have locally installed specific version of the package. Some of our conventions in Kibana rely on not using locally setup dependencies, but we relied on central dep management in the root.

This PR removes the local dependency

@delanni delanni added the release_note:skip Skip the PR/issue when compiling release notes label Apr 15, 2026
@delanni delanni requested a review from a team as a code owner April 15, 2026 12:01
@delanni delanni added the backport:skip This PR does not require backporting label Apr 15, 2026
@delanni delanni enabled auto-merge (squash) April 15, 2026 12:54
@elasticmachine
Copy link
Copy Markdown
Contributor

⏳ Build in-progress, with failures

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #131 / InfraOps App Metrics UI Hosts View #With data #Single Host Flyout Tabs Dashboards Tab should render dashboards tab splash screen with option to add dashboard
  • [job] [logs] affected Scout: [ platform / streams_app-stateful-classic ] plugin / local-stateful-classic - Stream data routing - AI suggestions editing validation - should show error when editing suggestion with duplicate name
  • [job] [logs] affected Scout: [ platform / streams_app-stateful-classic ] plugin / local-stateful-classic - Stream data routing - AI suggestions editing validation - should show error when editing suggestion with duplicate name

@delanni delanni disabled auto-merge April 15, 2026 15:50
@delanni delanni merged commit 365af6c into elastic:main Apr 15, 2026
16 of 17 checks passed
@juliaElastic
Copy link
Copy Markdown
Contributor

juliaElastic commented Apr 16, 2026

@delanni Can we backport this PR? The PR updating the dependency is backported to all open branches: #263105

Never mind, #252345 was not backported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes v9.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants