Skip to content
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

Bump typescript 4.9 #144469

Closed
wants to merge 26 commits into from
Closed

Bump typescript 4.9 #144469

wants to merge 26 commits into from

Conversation

jbudz
Copy link
Member

@jbudz jbudz commented Nov 2, 2022

WIP

@mistic
Copy link
Member

mistic commented Nov 9, 2022

@jbudz as part of this PR could you please take care of every TODO(jbudz) listed at #144841 (once it is merged) ? The things under the TODO comment will only working after upgrading TS to v4.8 but are definitely things we want to have.

@mistic mistic changed the title Bump typescript 4.8 Bump typescript 4.9 Dec 12, 2022
@kibana-ci
Copy link
Collaborator

kibana-ci commented Dec 22, 2022

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Security Solution Tests #4 / Add exception using data views from rule details Creates an exception item from alert actions overflow menu
  • [job] [logs] Jest Integration Tests #1 / indexExports() ambient types includes declarations for local ambient types, "ambientRefs" for globals
  • [job] [logs] Jest Integration Tests #1 / indexExports() export by type combining exports by value one value is exported twice and either is by value
  • [job] [logs] Jest Integration Tests #1 / indexExports() export references includes referenced declarations in locals
  • [job] [logs] Jest Integration Tests #1 / indexExports() finds references in importType.typeArguments exports by value one value is exported twice and either is by value
  • [job] [logs] Jest Integration Tests #1 / indexExports() ignores importTypes from node_modules but resolves local import types exports by value one value is exported twice and either is by value
  • [job] [logs] Jest Integration Tests #1 / indexExports() simple produces valid index
  • [job] [logs] Jest Integration Tests #1 / indexExports() type only exports exports by value one value is exported twice and either is by value
  • [job] [logs] Jest Integration Tests #1 / indexExports() undeclared symbols throws a helpful error when exported symbols are not found
  • [job] [logs] Security Solution Tests #4 / Row renderers Selected renderer can be disabled and enabled
  • [job] [logs] Jest Integration Tests #1 / summarizePackage() basic example produces expected type summary
  • [job] [logs] Security Solution Tests #4 / Timeline notes tab should contain notes

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
monitoring 452.1KB 452.1KB +22.0B

History

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

@jbudz jbudz closed this Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants