-
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
Upgrade Typescript to 4.3.5 #104470
Upgrade Typescript to 4.3.5 #104470
Conversation
@dgieselaar mind taking a look at this? Looks like you added last week.
|
@tylersmalley fixed the errors in |
@elasticmachine merge upstream |
1 similar comment
@elasticmachine merge upstream |
Signed-off-by: Tyler Smalley <[email protected]>
Signed-off-by: Tyler Smalley <[email protected]>
Signed-off-by: Tyler Smalley <[email protected]>
Someone from @elastic/kibana-core mind taking a look at these failures from bootstrap/build_ts_refs? view errors
|
|
||
/** | ||
* Http response headers to set. | ||
* @public | ||
*/ | ||
export type ResponseHeaders = | ||
| Record<KnownHeaders, string | string[]> |
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.
KnownHeaders
helper is broken, I didn't manage to fix it quickly. Its only purpose is to improve autocomplete in IDE, so I removed it for time being. I will fix it 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.
Ran into the same issue in Fleet. Just replacing with a string[]
for now.
const headersToProxy: KnownHeaders[] = ['content-type', 'cache-control']; |
@tylersmalley fixed the Core errors |
Signed-off-by: Tyler Smalley <[email protected]>
Signed-off-by: Tyler Smalley <[email protected]>
Signed-off-by: Tyler Smalley <[email protected]>
@elastic/kibana-app can you please assist with the following errors in your plugins while bootstrapping? Thanks view errors
|
@elastic/kibana-app-services can you please assist with the following errors in your plugins while bootstrapping? Thanks view errors
|
@elastic/kibana-presentation can you please assist with the following errors in your plugins while bootstrapping? Thanks view errors
|
@elastic/security-solution can you please assist with the following errors in your plugins while bootstrapping? Thanks view errors
|
view errors
|
view errors
|
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.
Fleet changes LGTM
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.
infra
plugin changes LGTM, thank you
Signed-off-by: Tyler Smalley <[email protected]>
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.
LGTM for threat hunting! thank you
Signed-off-by: Tyler Smalley <[email protected]>
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.
LGTM for data-discovery-team
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.
Uptime changes LGTM.
Signed-off-by: Tyler Smalley <[email protected]>
Signed-off-by: Tyler Smalley <[email protected]>
Signed-off-by: Tyler Smalley <[email protected]>
Signed-off-by: Tyler Smalley <[email protected]>
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.
Stuff breaks, but we'll follow up separately in #120272
💚 Build Succeeded
Metrics [docs]Public APIs missing comments
Async chunks
Public APIs missing exports
Page load bundle
Unknown metric groupsAPI count
References to deprecated APIs
History
To update your PR or re-run it, just comment with: |
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.
Stack Management changes LGTM:
- Console change is a code style change
- Mappings editor change removes unreachable code branch
- Ingest Pipelines change doesn't affect behavior (tested locally to verify)
- Upgrade Assistant test change retains coverage of behavior intended to be tested
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.
Thanks for addressing Thomas's comments!
Co-authored-by: Mikhail Shustov <[email protected]> Co-authored-by: Dario Gieselaar <[email protected]> Co-authored-by: CJ Cenizal <[email protected]> Co-authored-by: Justin Kambic <[email protected]> Co-authored-by: Stratoula Kalafateli <[email protected]> Co-authored-by: Jean-Louis Leysens <[email protected]> Co-authored-by: Kyle Pollich <[email protected]> Co-authored-by: Xavier Mouligneau <[email protected]> Co-authored-by: Kevin Qualters <[email protected]> Co-authored-by: Candace Park <[email protected]> Co-authored-by: Brian Seeders <[email protected]> Co-authored-by: spalger <[email protected]>
Co-authored-by: Mikhail Shustov <[email protected]> Co-authored-by: Dario Gieselaar <[email protected]> Co-authored-by: CJ Cenizal <[email protected]> Co-authored-by: Justin Kambic <[email protected]> Co-authored-by: Stratoula Kalafateli <[email protected]> Co-authored-by: Jean-Louis Leysens <[email protected]> Co-authored-by: Kyle Pollich <[email protected]> Co-authored-by: Xavier Mouligneau <[email protected]> Co-authored-by: Kevin Qualters <[email protected]> Co-authored-by: Candace Park <[email protected]> Co-authored-by: Brian Seeders <[email protected]> Co-authored-by: spalger <[email protected]> (cherry picked from commit 92a8636) # Conflicts: # package.json # packages/kbn-alerts/BUILD.bazel # packages/kbn-babel-register/BUILD.bazel # packages/kbn-babel-register/tsconfig.json # packages/kbn-pm/dist/index.js # packages/kbn-test/src/jest/run.ts # src/core/public/public.api.md # src/core/public/theme/theme_service.ts # src/core/types/elasticsearch/search.ts # src/plugins/discover/public/build_services.ts # src/plugins/presentation_util/public/components/controls/control_group/embeddable/control_group_container_factory.ts # x-pack/plugins/apm/server/lib/helpers/create_es_client/create_apm_event_client/index.ts # x-pack/plugins/apm/server/lib/helpers/setup_request.ts # x-pack/plugins/apm/server/lib/services/get_service_error_groups/get_service_error_group_main_statistics.ts # x-pack/plugins/apm/server/lib/transaction_groups/get_transaction_group_stats.ts # x-pack/plugins/apm/server/types.ts # x-pack/plugins/data_visualizer/public/application/index_data_visualizer/utils/saved_search_utils.test.ts # x-pack/plugins/enterprise_search/public/applications/app_search/components/crawler/components/crawl_details_flyout/accordion_list.test.tsx # x-pack/plugins/enterprise_search/public/applications/app_search/components/crawler/components/crawl_requests_table.test.tsx # x-pack/plugins/enterprise_search/public/applications/app_search/components/crawler/components/crawl_requests_table.tsx # x-pack/plugins/infra/public/alerting/log_threshold/components/expression_editor/criterion.tsx # x-pack/plugins/ingest_pipelines/public/application/sections/pipelines_create/pipelines_create.tsx # x-pack/plugins/rule_registry/server/utils/create_lifecycle_rule_type.test.ts # x-pack/plugins/security_solution/public/detections/pages/detection_engine/rules/helpers.test.tsx # x-pack/plugins/security_solution/server/lib/detection_engine/routes/signals/query_signals_route.test.ts # x-pack/plugins/security_solution/server/lib/detection_engine/signals/utils.test.ts # x-pack/plugins/uptime/public/state/effects/journey.ts # x-pack/plugins/uptime/server/lib/requests/get_ping_histogram.ts # yarn.lock
# Backport This will backport the following commits from `main` to `7.17`: - [Upgrade Typescript to 4.3.5 (#104470)](#104470) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Tyler Smalley","email":"[email protected]"},"sourceCommit":{"committedDate":"2021-12-03T00:27:41Z","message":"Upgrade Typescript to 4.3.5 (#104470)\n\nCo-authored-by: Mikhail Shustov <[email protected]>\r\nCo-authored-by: Dario Gieselaar <[email protected]>\r\nCo-authored-by: CJ Cenizal <[email protected]>\r\nCo-authored-by: Justin Kambic <[email protected]>\r\nCo-authored-by: Stratoula Kalafateli <[email protected]>\r\nCo-authored-by: Jean-Louis Leysens <[email protected]>\r\nCo-authored-by: Kyle Pollich <[email protected]>\r\nCo-authored-by: Xavier Mouligneau <[email protected]>\r\nCo-authored-by: Kevin Qualters <[email protected]>\r\nCo-authored-by: Candace Park <[email protected]>\r\nCo-authored-by: Brian Seeders <[email protected]>\r\nCo-authored-by: spalger <[email protected]>","sha":"92a8636f0ff63ab072527574e96e6616327b2ea4","branchLabelMapping":{"^v8.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Operations","Feature:Embedding","Team:APM","Feature:ExpressionLanguage","Team:uptime","release_note:skip","backport:skip","Feature:Drilldowns","Team:Fleet","v8.1.0","ci:all-cypress-suites"],"number":104470,"url":"https://github.com/elastic/kibana/pull/104470","mergeCommit":{"message":"Upgrade Typescript to 4.3.5 (#104470)\n\nCo-authored-by: Mikhail Shustov <[email protected]>\r\nCo-authored-by: Dario Gieselaar <[email protected]>\r\nCo-authored-by: CJ Cenizal <[email protected]>\r\nCo-authored-by: Justin Kambic <[email protected]>\r\nCo-authored-by: Stratoula Kalafateli <[email protected]>\r\nCo-authored-by: Jean-Louis Leysens <[email protected]>\r\nCo-authored-by: Kyle Pollich <[email protected]>\r\nCo-authored-by: Xavier Mouligneau <[email protected]>\r\nCo-authored-by: Kevin Qualters <[email protected]>\r\nCo-authored-by: Candace Park <[email protected]>\r\nCo-authored-by: Brian Seeders <[email protected]>\r\nCo-authored-by: spalger <[email protected]>","sha":"92a8636f0ff63ab072527574e96e6616327b2ea4"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.1.0","labelRegex":"^v8.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/104470","number":104470,"mergeCommit":{"message":"Upgrade Typescript to 4.3.5 (#104470)\n\nCo-authored-by: Mikhail Shustov <[email protected]>\r\nCo-authored-by: Dario Gieselaar <[email protected]>\r\nCo-authored-by: CJ Cenizal <[email protected]>\r\nCo-authored-by: Justin Kambic <[email protected]>\r\nCo-authored-by: Stratoula Kalafateli <[email protected]>\r\nCo-authored-by: Jean-Louis Leysens <[email protected]>\r\nCo-authored-by: Kyle Pollich <[email protected]>\r\nCo-authored-by: Xavier Mouligneau <[email protected]>\r\nCo-authored-by: Kevin Qualters <[email protected]>\r\nCo-authored-by: Candace Park <[email protected]>\r\nCo-authored-by: Brian Seeders <[email protected]>\r\nCo-authored-by: spalger <[email protected]>","sha":"92a8636f0ff63ab072527574e96e6616327b2ea4"}}]}] BACKPORT--> --------- Co-authored-by: Tyler Smalley <[email protected]>
Closes #100563
Blockers:
PRs:
Release Notes:
microsoft/TypeScript#43194
TODO after merge: