-
Notifications
You must be signed in to change notification settings - Fork 8.5k
Create the @kbn/std package
#77329
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
Create the @kbn/std package
#77329
Conversation
|
Pinging @elastic/kibana-platform (Team:Platform) |
mikecote
left a comment
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.
Alerts and Actions plugin code LGTM
|
Pinging @elastic/ingest-management (Team:Ingest Management) |
legrego
left a comment
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.
Security changes LGTM
jfsiii
left a comment
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 Ingest Management
lukeelmers
left a comment
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.
App arch changes LGTM
stratoula
left a comment
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.
Kibana App Changes LGTM
watson
left a comment
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.
@elastic/kibana-security changes LGTM 👍
💛 Build succeeded, but was flaky
Test FailuresChrome X-Pack UI Functional Tests.x-pack/test/functional_with_es_ssl/apps/triggers_actions_ui/alerts·ts.Actions and Triggers app alerts should delete all selectionStandard OutStack TraceBuild metrics@kbn/optimizer bundle module count
async chunks size
page load bundle size
distributable file count
History
To update your PR or re-run it, just comment with: |
* move `src/core/utils` to the @kbn/std package * update README * update codeowners and add yarn lock file * remove export from src/core/public and src/core/server and use package import instead * update generated doc * adapt forgotten import * update `data` plugin doc # Conflicts: # .github/CODEOWNERS # src/core/public/application/capabilities/capabilities_service.mock.ts # src/core/public/index.ts
* move `src/core/utils` to the @kbn/std package * update README * update codeowners and add yarn lock file * remove export from src/core/public and src/core/server and use package import instead * update generated doc * adapt forgotten import * update `data` plugin doc # Conflicts: # .github/CODEOWNERS # src/core/public/application/capabilities/capabilities_service.mock.ts # src/core/public/index.ts
Summary
Fix #76925
src/core/utilsto the new@kbn/stdpackagesrc/core/serverandsrc/core/publicChecklist