Skip to content

[7.x] chore(NA): moving @kbn/ui-shared-deps into bazel (#101669)#102666

Merged
mistic merged 2 commits intoelastic:7.xfrom
mistic:backport/7.x/pr-101669
Jun 19, 2021
Merged

[7.x] chore(NA): moving @kbn/ui-shared-deps into bazel (#101669)#102666
mistic merged 2 commits intoelastic:7.xfrom
mistic:backport/7.x/pr-101669

Conversation

@mistic
Copy link
Copy Markdown
Contributor

@mistic mistic commented Jun 18, 2021

Backports the following commits to 7.x:

* chore(NA): moving @kbn/io-ts-utils into bazel

* chore(NA): moving @kbn/ui-shared-deps into bazel

* chore(NA): compelte working build for @kbn/ui-shared-deps

* chore(NA): solve eslint problems

* chore(NA): solve typechecking

* chore(NA): debugger changes

* chore(NA): update optimizer basic integration tests

* chore(NA): ship kbn/ui-shared-deps metrics.json from new location at shared_built_assets

* chore(NA): use correct ui-shared-deps metrics file location

* chore(NA): remove webpack bazel config

* chore(NA): implement improvements on webpack config

* chore(NA): remove extra comment

* chore(NA): try esbuild-loader minimizer

* Revert "chore(NA): try esbuild-loader minimizer"

This reverts commit bffc49a.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
# Conflicts:
#	test/scripts/jenkins_build_kibana.sh
@mistic mistic added the backport This PR is a backport of another PR label Jun 18, 2021
@mistic mistic enabled auto-merge (squash) June 18, 2021 19:27
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Page load bundle

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

id before after diff
kbnUiSharedDeps-css 503.7KB 504.2KB +429.0B
kbnUiSharedDeps-js 6.4MB 6.4MB -6.0B
total +423.0B

History

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

@mistic mistic merged commit 3cfbe71 into elastic:7.x Jun 19, 2021
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants