kibana icon indicating copy to clipboard operation
kibana copied to clipboard

chore(NA): upgrade typescript into v4.9.5

Open mistic opened this issue 1 year ago • 3 comments

This PR bumps the Typescript version used on Kibana into v4.9.5. The full set of changes can be found here. So far, as long I can see, the performance is better than in the version we're currently in.

I didn't want to make assumptions for such a diversity of type errors across the codebase so instead I choose to mark the failures with @ts-expect-error and let each team decide how to handle it.

There is a list below with the files (and teams those belong to) where the annotations were added. If each team could provide help of fixing the ones under their domain it would be fantastic. It can be done in this PR or in subsequent ones.

Here it goes the list for follow up:

@elastic/kibana-core - tracking issue: https://github.com/elastic/kibana/issues/176153

@elastic/kibana-presentation

@elastic/apm

@elastic/obs-knowledge-team

@elastic/security-detection-engine

@elastic/appex-sharedux

@elastic/kibana-visualizations

@elastic/kibana-data-discovery

@elastic/platform-deployment-management

@elastic/kibana-security

@elastic/ml-ui

@elastic/response-ops

@elastic/security-threat-hunting-investigations

@elastic/security-generative-ai

@elastic/enterprise-search-frontend

@elastic/fleet

@elastic/platform-deployment-management

@elastic/obs-ux-logs-team / @elastic/obs-ux-infra_services-team

@elastic/kibana-gis

@elastic/observability-ui

@elastic/security-defend-workflows

@elastic/security-solution

mistic avatar Jan 20 '24 19:01 mistic

Pinging @elastic/kibana-operations (Team:Operations)

elasticmachine avatar Feb 01 '24 18:02 elasticmachine

@banderror as I mentioned in the PR description, feel free to open a PR against my branch and I will merge it into this one or follow up in a new one after this gets merged 😃

mistic avatar Feb 05 '24 15:02 mistic

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

elasticmachine avatar Feb 23 '24 20:02 elasticmachine

:yellow_heart: Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Defend Workflows Cypress Tests #5 / Response console Host Isolation: should isolate a host from response console should isolate a host from response console

Metrics [docs]

Async chunks

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

id before after diff
expressionLegacyMetricVis 11.9KB 11.9KB -10.0B
visTypeTimeseries 514.3KB 514.3KB +8.0B
total -2.0B

Public APIs missing exports

Total count of every type that is part of your API that should be exported but is not. This will cause broken links in the API documentation system. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats exports for more detailed information.

id before after diff
observabilityShared 19 20 +1

History

  • :yellow_heart: Build #195379 was flaky 8b3f4826203319656c7ec489edae52de798257f7
  • :broken_heart: Build #195377 failed 3411e28a81f82d841a5f3a4dc007e5ed9f89f12b
  • :broken_heart: Build #195366 failed e9fe384d2962fba85146d3ffc89c2dd0855f974b
  • :broken_heart: Build #195362 failed 3a8ba61f53b4f4c341a1150c4f7bfdfa2c709889
  • :broken_heart: Build #194104 failed 177f8c99bb67142e95b4cb20ad68487c2f7c041d
  • :broken_heart: Build #193993 failed 8789f21161d26d494eb7bc80d40c1b2437848020

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

kibana-ci avatar Feb 24 '24 20:02 kibana-ci

I've went ahead and merged this one. The missing places with a ts-expect-error to be fixed can be found in the description of the PR. Please let me know if you have any other feedback.

mistic avatar Feb 25 '24 01:02 mistic

Cross-referencing #160033 for discoverability

delanni avatar Mar 06 '24 14:03 delanni