-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[TypeScript] Upgrade to 4.5.3 #120812
[TypeScript] Upgrade to 4.5.3 #120812
Conversation
be1ec3c
to
565c1d6
Compare
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.
Presentation team changes LGTM! Thank you for ignoring the TS errors. I have created an issue to track our TODO to update those files.
35% faster?! Yes, please! Running: ts-4-5-2:
main:
|
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.
🎉
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
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.
OK for the Core changes.
Please note the current PR updates TS to v4.5 but developers can't use some new TypeScript features until other code analysis tools are updated to versions with TypeScript v4.5 support:
https://github.com/typescript-eslint/typescript-eslint/releases/tag/v5.6.0
https://github.com/babel/babel/releases/tag/v7.16.0
https://prettier.io/blog/2021/11/25/2.5.0.html
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 change 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.
Transform change 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.
I don't see any of Stack Management plugins changed so approving to unblock the merging
@@ -23,7 +23,9 @@ export const AutoDownload: React.FC<AutoDownloadProps> = ({ blob, name, onDownlo | |||
|
|||
useEffect(() => { | |||
if (blob && anchorRef?.current) { | |||
// @ts-expect-error | |||
if (typeof window.navigator.msSaveOrOpenBlob === 'function') { |
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.
Unrelated to the PR: we should probably not ship this code anymore since it's an API that was just supported by Internet Explorer. (takes a mental note to create a follow-up PR)
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.
👍
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!
💚 Build Succeeded
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
(cherry picked from commit 16af3e6) # Conflicts: # package.json # src/plugins/embeddable/public/lib/attribute_service/attribute_service.tsx # x-pack/plugins/canvas/public/components/workpad_header/editor_menu/editor_menu.tsx # x-pack/plugins/security_solution/public/management/pages/policy/store/policy_details/middleware/policy_trusted_apps_middleware.ts
# Backport This will backport the following commits from `main` to `7.17`: - [[TypeScript] Upgrade to 4.5.3 (#120812)](#120812) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Brian Seeders","email":"brian.seeders@elastic.co"},"sourceCommit":{"committedDate":"2021-12-13T17:54:17Z","message":"[TypeScript] Upgrade to 4.5.3 (#120812)","sha":"16af3e612d27bd3608787a3daab121d648787f67","branchLabelMapping":{"^v8.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Operations","Feature:Embedding","Team:APM","release_note:skip","Feature:CI","backport:skip","Team:Fleet","v8.1.0"],"number":120812,"url":"https://github.com/elastic/kibana/pull/120812","mergeCommit":{"message":"[TypeScript] Upgrade to 4.5.3 (#120812)","sha":"16af3e612d27bd3608787a3daab121d648787f67"}},"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/120812","number":120812,"mergeCommit":{"message":"[TypeScript] Upgrade to 4.5.3 (#120812)","sha":"16af3e612d27bd3608787a3daab121d648787f67"}}]}] BACKPORT--> Co-authored-by: Brian Seeders <brian.seeders@elastic.co>
Upgrades TypeScript to the latest version.
See #121003 for the
isEditable
issue