-
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
[Logs+] Implement Logs Dataset selector #159907
Merged
tonyghiani
merged 142 commits into
elastic:main
from
tonyghiani:2655-implement-log-data-stream-selector-rebased
Jun 28, 2023
Merged
Changes from 123 commits
Commits
Show all changes
142 commits
Select commit
Hold shift + click to select a range
9ab4529
feat(observability-logs): create plugin boilerplate
6bcf4f8
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine eeecf6f
[CI] Auto-commit changed files from 'node scripts/lint_ts_projects --…
kibanamachine eea7560
[CI] Auto-commit changed files from 'node scripts/generate codeowners'
kibanamachine 1a6fb0b
[CI] Auto-commit changed files from 'node scripts/build_plugin_list_d…
kibanamachine d2a76bb
feat(observability-logs): try discover customization
559bc9c
[CI] Auto-commit changed files from 'node scripts/build_plugin_list_d…
kibanamachine c775f49
feat(observability-logs): update profileId
5d957b5
feat(observability-logs): add lazy loading and wip component
b68f33b
feat(observability-logs): add story for data stream selector
826cb03
feat(observability-logs): create nested panels
7ed6317
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine 4d2d8cd
[CI] Auto-commit changed files from 'node scripts/lint_ts_projects --…
kibanamachine 6660674
feat(observability-logs): apply customization to hide discover controls
c2bd77e
feat(observability-logs): add integration types
80b4dc2
feat(observability-logs): update integrations tree
cc23a2a
refactor(observability-logs): remove temporary index icon from data v…
d102bdb
feat(observability-logs): create integrations state machine folder
60436ed
[CI] Auto-commit changed files from 'node scripts/lint_ts_projects --…
kibanamachine bc227c2
refactor(observability-log): allow ad-hoc data view creation with specs
ee42d87
chore(observability-log): fix jest config
dd530db
chore(observability-log): add optimizer bundle limit
ec969ef
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine b39b901
feat(observability-log): draft integrations state machine
2220c29
feat(observability-log): adding integrations service
dd77c92
[CI] Auto-commit changed files from 'node scripts/precommit_hook.js -…
kibanamachine b3cb323
[CI] Auto-commit changed files from 'node scripts/lint_ts_projects --…
kibanamachine 790df31
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine c47afcc
feat(observability-log): use integration service
01fb593
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine 99fbea3
[CI] Auto-commit changed files from 'node scripts/lint_ts_projects --…
kibanamachine 5cf58f4
refactor(observability-log): rename integrations to data streams service
8391f03
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine fc535df
refactor(observability-log): move state machine actions
e549917
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine 0ad11d1
refactor(observability-log): load more feature
0b8fdc7
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine 75f92c9
feat(infra): handle integrations search and load more guard
a5feafd
feat(infra): update url creator
9d050fb
feat(observability_logs): add search and cache
8a14ec7
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine 0d31cc6
refactor(observability_logs): handle missing uncategorized streams
d5e10df
feat(observability_logs): add data stream fetching and rendering logic
7aef4ca
[CI] Auto-commit changed files from 'node scripts/precommit_hook.js -…
kibanamachine 02bd234
feat(observability_logs): implement dynamic and lazy load content
a00b747
feat(observability_logs): update local search
eca8902
feat(observability_logs): renames
3131c30
feat(observability_logs): update DataStreamSelector story
c8036c6
refactor(observability_logs): reorganize code
4bf3fa5
refactor(observability_logs): update search strategy
9d862ab
refactor(observability_logs): store initial search in cache
48ceaae
refactor(observability_logs): integrations status
e0b61ac
[CI] Auto-commit changed files from 'node scripts/precommit_hook.js -…
kibanamachine d8e3a31
refactor(observability_logs): improve typestates
9aee3b1
refactor(observability_logs): begin data stream selector state refactor
fc44fb6
refactor(observability_logs): wip on selector state machine
6a586a9
[CI] Auto-commit changed files from 'node scripts/precommit_hook.js -…
kibanamachine 9ac5732
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine 7211653
refactor(observability_logs): create internal state machine and split…
2b4f3e5
refactor(observability_logs): remove conflicting code
126ea3b
refactor(observability_logs): update data streams machine
474098e
refactor(observability_logs): update state machine restore search
e7e36e4
refactor(observability_logs): update storybook
bc0a6a2
refactor(observability_logs): small adjustments
8e884f0
refactor(observability_logs): minor changes
2a1495d
[CI] Auto-commit changed files from 'node scripts/precommit_hook.js -…
kibanamachine 1e61e77
refactor(discover-log-explorer): rename plugin
7ef8c35
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine 9654ba1
Merge branch 'main' into 2655-implement-log-data-stream-selector-rebased
tonyghiani 3bda2aa
refactor(discover-log-explorer): update missing renaming
bf1a324
refactor(discover-log-explorer): remove merge leftover
59a68f7
refactor(discover-log-explorer): remove server files
5e7a431
[CI] Auto-commit changed files from 'node scripts/lint_ts_projects --…
kibanamachine 106e3a9
[CI] Auto-commit changed files from 'node scripts/generate codeowners'
kibanamachine b498853
[CI] Auto-commit changed files from 'node scripts/build_plugin_list_d…
kibanamachine 073d301
refactor(discover-log-explorer): update exports
7a713e3
refactor(discover-log-explorer): track plugin i18n
c4fd397
refactor(discover-log-explorer): fix linting errors
e70e44c
refactor(discover-log-explorer): renaming to dataset
14614ee
Merge branch 'main' into 2655-implement-log-data-stream-selector-rebased
tonyghiani 5228ce2
refactor(discover-log-explorer): updat comment
3e0b492
refactor(discover-log-explorer): update limits
04fa5a9
refactor(discover-log-explorer): update types issues
906e7cd
refactor(discover-log-explorer): fix local search value
4054fc6
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine f10030e
refactor(discover-log-explorer): fix missing test
ad5b2d5
Merge branch '2655-implement-log-data-stream-selector-rebased' of git…
b6b654b
refactor(discover-log-explorer): add fake test
32e8580
refactor(discover-log-explorer): remove fake test
1cc0d05
refactor(discover-log-explorer): reduce exports
3aebf92
Merge branch 'main' into 2655-implement-log-data-stream-selector-rebased
tonyghiani 4b98e27
refactor(discover-log-explorer): apply first changes
8109cc0
Merge branch 'main' into 2655-implement-log-data-stream-selector-rebased
stratoula 6f87200
Merge branch 'main' into 2655-implement-log-data-stream-selector-rebased
tonyghiani b39f91c
fix(discover-log-explorer): routes usage
0818d3f
Merge branch 'main' into 2655-implement-log-data-stream-selector-rebased
tonyghiani 19e9be2
[CI] Auto-commit changed files from 'node scripts/precommit_hook.js -…
kibanamachine 9caebee
docs(discover-log-explorer): update readme with quick introduction
8a5e765
Merge branch 'main' into 2655-implement-log-data-stream-selector-rebased
tonyghiani 20f4363
refactor(discover-log-explorer): reduce by ~80% initial plugin page l…
5c2b3c8
refactor(discover-log-explorer): abstract Integration and Dataset log…
3a06731
refactor(discover-log-explorer): remove mock data
f2aa421
refactor(discover-log-explorer): remove title name prefix
d2a4431
refactor(discover-log-explorer): minor changes
93d3537
refactor(discover-log-explorer): encode type for http request
cf48f3c
[CI] Auto-commit changed files from 'node scripts/lint_ts_projects --…
kibanamachine aedf1bb
refactor(kbn-io-ts-utils): remove redundant type
007b64b
Merge branch '2655-implement-log-data-stream-selector-rebased' of git…
4462f41
refactor(discover-log-explorer): remove unused type
e950984
refactor(discover-log-explorer): use dataset id
645f3af
refactor(discover-log-explorer): update toSpec method signature
d4df21d
[CI] Auto-commit changed files from 'node scripts/lint_ts_projects --…
kibanamachine 51db3a1
Merge branch 'main' into 2655-implement-log-data-stream-selector-rebased
tonyghiani 455e53f
refactor(discover-log-explorer): change requests
bfaa5bc
refactor(discover-log-explorer): rename dataStreams for consistency
6c8b461
Merge branch 'main' into 2655-implement-log-data-stream-selector-rebased
tonyghiani fa90bff
[CI] Auto-commit changed files from 'node scripts/lint_ts_projects --…
kibanamachine 3c0acf7
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine 0a0d34a
refactor(discover-log-explorer): remove unused import
1351408
Merge branch '2655-implement-log-data-stream-selector-rebased' of git…
48bdc91
refactor(discover-log-explorer): fix missing encoding
ee0eeed
refactor(discover-log-explorer): update change requests
549cea8
refactor(discover-log-explorer): update params conversion
57aee54
refactor(discover-log-explorer): move dynamic to public/utils folder
eafd4b9
refactor(discover-log-explorer): update cache with LRU and hashObject
294602b
refactor(discover-log-explorer): update cache name
2bc90cd
refactor(discover-log-explorer): apply suggestion for entity list
e583a6f
refactor(discover-log-explorer): rename cache file
d8ff061
Merge branch 'main' into 2655-implement-log-data-stream-selector-rebased
tonyghiani cfc5019
Update x-pack/plugins/discover_log_explorer/public/state_machines/int…
tonyghiani 7a4dcd7
refactor(discover-log-explorer): update DoneEvent type
29f0d7b
Merge branch 'main' into 2655-implement-log-data-stream-selector-rebased
tonyghiani e8ec06e
test(discover-log-explorer): add basic test to assert customizations …
edcb90e
refactor(discover-log-explorer): switch to pure action for maybeResto…
0ffdf0a
refactor(discover-log-explorer): move spyRef into buildIntegrationsTr…
389c097
refactor(discover-log-explorer): remove import
7d605a7
refactor(discover-log-explorer): add explicit close action
8184fb9
Update x-pack/test/functional/apps/discover_log_explorer/customizatio…
tonyghiani cbba119
refactor(discover-log-explorer): add copies changes ans move conditional
b214654
Merge branch 'main' into 2655-implement-log-data-stream-selector-rebased
tonyghiani 033af2d
refactor(discover-log-explorer): add storybook alias to buildkite
2893978
Merge branch 'main' into 2655-implement-log-data-stream-selector-rebased
tonyghiani File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Validating CODEOWNERS rules …
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
11 changes: 11 additions & 0 deletions
11
x-pack/plugins/discover_log_explorer/.storybook/__mocks__/package_icon.tsx
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the Elastic License | ||
* 2.0; you may not use this file except in compliance with the Elastic License | ||
* 2.0. | ||
*/ | ||
import React from 'react'; | ||
import { EuiIcon } from '@elastic/eui'; | ||
|
||
// Export mock package icon that doesn't trigger http requests | ||
export const PackageIcon = () => <EuiIcon type="package" style={{ marginRight: 8 }} />; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the Elastic License | ||
* 2.0; you may not use this file except in compliance with the Elastic License | ||
* 2.0. | ||
*/ | ||
|
||
const defaultConfig = require('@kbn/storybook').defaultConfig; | ||
|
||
module.exports = { | ||
...defaultConfig, | ||
stories: ['../**/*.stories.mdx', ...defaultConfig.stories], | ||
webpackFinal: async (config) => { | ||
const originalConfig = await defaultConfig.webpackFinal(config); | ||
|
||
// Mock fleet plugin for PackageIcon component | ||
originalConfig.resolve.alias['@kbn/fleet-plugin/public'] = require.resolve( | ||
'./__mocks__/package_icon' | ||
); | ||
return originalConfig; | ||
}, | ||
}; |
13 changes: 13 additions & 0 deletions
13
x-pack/plugins/discover_log_explorer/.storybook/preview.js
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the Elastic License | ||
* 2.0; you may not use this file except in compliance with the Elastic License | ||
* 2.0. | ||
*/ | ||
export const parameters = { | ||
docs: { | ||
source: { | ||
type: 'code', // without this, stories in mdx documents freeze the browser | ||
}, | ||
}, | ||
}; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
# Discover Log Explorer | ||
|
||
This plugin registers a `log-explorer` profile using the Discover customization framework, offering several affordances specifically designed for log consumption. | ||
|
||
The plugin enhances the capabilities of Discover in the following ways: | ||
|
||
- **Dataset selector**: this customization on the Discover page replaces the DataViews picker with a Logs dataset selector built ad-hoc to provide a better experience when navigating throught all the available datasets. | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the Elastic License | ||
* 2.0; you may not use this file except in compliance with the Elastic License | ||
* 2.0. | ||
*/ | ||
|
||
export const LOG_EXPLORER_PROFILE_ID = 'log-explorer'; |
24 changes: 24 additions & 0 deletions
24
x-pack/plugins/discover_log_explorer/common/datasets/errors.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the Elastic License | ||
* 2.0; you may not use this file except in compliance with the Elastic License | ||
* 2.0. | ||
*/ | ||
|
||
/* eslint-disable max-classes-per-file */ | ||
|
||
export class FindIntegrationsError extends Error { | ||
constructor(message: string) { | ||
super(message); | ||
Object.setPrototypeOf(this, new.target.prototype); | ||
this.name = 'FindIntegrationsError'; | ||
} | ||
} | ||
|
||
export class FindDatasetsError extends Error { | ||
constructor(message: string) { | ||
super(message); | ||
Object.setPrototypeOf(this, new.target.prototype); | ||
this.name = 'FindDatasetsError'; | ||
} | ||
} |
10 changes: 10 additions & 0 deletions
10
x-pack/plugins/discover_log_explorer/common/datasets/index.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the Elastic License | ||
* 2.0; you may not use this file except in compliance with the Elastic License | ||
* 2.0. | ||
*/ | ||
|
||
export * from './types'; | ||
export { Dataset } from './models/dataset'; | ||
export { Integration } from './models/integration'; |
41 changes: 41 additions & 0 deletions
41
x-pack/plugins/discover_log_explorer/common/datasets/models/dataset.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the Elastic License | ||
* 2.0; you may not use this file except in compliance with the Elastic License | ||
* 2.0. | ||
*/ | ||
|
||
import { DataViewSpec } from '@kbn/data-views-plugin/common'; | ||
import { DatasetId, DatasetType, IntegrationType } from '../types'; | ||
|
||
type IntegrationBase = Pick<IntegrationType, 'name' | 'version'>; | ||
|
||
export class Dataset { | ||
id: DatasetId; | ||
name: DatasetType['name']; | ||
title: DatasetType['title']; | ||
parentIntegration?: IntegrationBase; | ||
|
||
private constructor(dataset: DatasetType, parentIntegration?: IntegrationType) { | ||
this.id = `dataset-${dataset.name}` as DatasetId; | ||
this.name = dataset.name; | ||
this.title = dataset.title ?? dataset.name; | ||
this.parentIntegration = parentIntegration && { | ||
name: parentIntegration.name, | ||
version: parentIntegration.version, | ||
}; | ||
} | ||
|
||
toDataviewSpec(): DataViewSpec { | ||
// Invert the property because the API returns the index pattern as `name` and a readable name as `title` | ||
return { | ||
id: this.id, | ||
name: this.title, | ||
title: this.name, | ||
}; | ||
} | ||
|
||
public static create(dataset: DatasetType, parentIntegration?: IntegrationType) { | ||
return new Dataset(dataset, parentIntegration); | ||
} | ||
} |
33 changes: 33 additions & 0 deletions
33
x-pack/plugins/discover_log_explorer/common/datasets/models/integration.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the Elastic License | ||
* 2.0; you may not use this file except in compliance with the Elastic License | ||
* 2.0. | ||
*/ | ||
|
||
import { Dataset } from './dataset'; | ||
import { IntegrationId, IntegrationType } from '../types'; | ||
|
||
export class Integration { | ||
id: IntegrationId; | ||
name: IntegrationType['name']; | ||
status: IntegrationType['status']; | ||
version: IntegrationType['version']; | ||
datasets: Dataset[]; | ||
|
||
private constructor(integration: Integration) { | ||
this.id = integration.id; | ||
this.name = integration.name; | ||
this.status = integration.status; | ||
this.version = integration.version; | ||
this.datasets = integration.datasets; | ||
} | ||
|
||
public static create(integration: IntegrationType) { | ||
return new Integration({ | ||
...integration, | ||
id: `integration-${integration.name}-${integration.version}` as IntegrationId, | ||
datasets: integration.dataStreams.map((dataset) => Dataset.create(dataset, integration)), | ||
}); | ||
} | ||
} |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Could we also add it to
.buildkite/scripts/steps/storybooks/build_and_upload.ts
as recently documented in #160473?