Skip to content

[8.15] [Security solution] Assistant package assistant/index.tsx cleanup (#190151)#191501

Merged
stephmilovic merged 2 commits intoelastic:8.15from
stephmilovic:backport/8.15/pr-190151
Aug 27, 2024
Merged

[8.15] [Security solution] Assistant package assistant/index.tsx cleanup (#190151)#191501
stephmilovic merged 2 commits intoelastic:8.15from
stephmilovic:backport/8.15/pr-190151

Conversation

@stephmilovic
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.15:

Questions ?

Please refer to the Backport tool documentation

…lastic#190151)

(cherry picked from commit 730f8ea)

# Conflicts:
#	x-pack/plugins/translations/translations/fr-FR.json
#	x-pack/plugins/translations/translations/ja-JP.json
#	x-pack/plugins/translations/translations/zh-CN.json
@stephmilovic stephmilovic added the backport This PR is a backport of another PR label Aug 27, 2024
@stephmilovic stephmilovic enabled auto-merge (squash) August 27, 2024 13:51
@kibana-ci
Copy link
Copy Markdown

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
integrationAssistant 538 544 +6
securitySolution 5616 5622 +6
total +12

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
integrationAssistant 938.3KB 937.6KB -679.0B
securitySolution 17.2MB 16.2MB -989.5KB
total -990.2KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
securitySolution 86.2KB 86.1KB -7.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
@kbn/elastic-assistant 16 15 -1

Total ESLint disabled count

id before after diff
@kbn/elastic-assistant 17 16 -1

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@stephmilovic stephmilovic merged commit 2e0658a into elastic:8.15 Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR release_note:fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants