Cleanup outdated @elastic/elasticsearch client type errors #101741
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mshustov
added
chore
release_note:skip
botelastic
bot
added
the
Team:Fleet
|
Pinging @elastic/fleet (Team:Fleet) |
cjcenizal
approved these changes
Jun 9, 2021
YulNaumenko
approved these changes
Jun 9, 2021
pgayvallet
approved these changes
Jun 9, 2021
peteharverson
approved these changes
Jun 10, 2021
flash1293
approved these changes
Jun 10, 2021
weltenwort
approved these changes
Jun 10, 2021
estermv
approved these changes
Jun 21, 2021
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
💔 Backport failed
To backport manually run: |
mshustov
added a commit
to mshustov/kibana
that referenced
this pull request
Jun 21, 2021
…01741) * fix errors and update comments in Core * fix errors or update comments in Security plugin * update spaces test * update task_manager files * update comments in monitoring plugin * fix errors in update comments in security_solutions * fix errors and update comments in data_enhanced * update fleet code * update infra code * update comment in trigger_actions_ui * update comment in lens * update comments in ES-UI code * update typings for search * update monitoring * remove outdated export # Conflicts: # x-pack/plugins/infra/server/lib/alerting/inventory_metric_threshold/evaluate_condition.ts # x-pack/plugins/snapshot_restore/server/routes/api/snapshots.ts
mshustov
added a commit
that referenced
this pull request
Jun 21, 2021
…1741) (#102747) * Cleanup outdated @elastic/elasticsearch client type errors (#101741) * fix errors and update comments in Core * fix errors or update comments in Security plugin * update spaces test * update task_manager files * update comments in monitoring plugin * fix errors in update comments in security_solutions * fix errors and update comments in data_enhanced * update fleet code * update infra code * update comment in trigger_actions_ui * update comment in lens * update comments in ES-UI code * update typings for search * update monitoring * remove outdated export # Conflicts: # x-pack/plugins/infra/server/lib/alerting/inventory_metric_threshold/evaluate_condition.ts # x-pack/plugins/snapshot_restore/server/routes/api/snapshots.ts * fix a result of wrong conflict resolution
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Follow up on #98266
Reduced the number of
@ts-expect-error @elastic/elasticsearchfrom 111 to 65.@ts-expect-error @elastic/elasticsearchdescriptions. You can find the full list of known problems here. These problems should be fixed in the following versions of the@elastic/elasticsearchlibrary.@elastic/elasticsearchfrom@ts-expect-errorwhen owners need to take actions to clean up the code (in separate PRs)