-
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
[kbn-es/installers] convert to TypeScript #120560
Conversation
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
Pinging @elastic/kibana-operations (Team:Operations) |
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 tested the available install commands and reviewed source changes. LGTM
* [kbn-es/installers] convert to TypeScript * catch invalid urls passed to installArchive()
💔 Backport failed
Successful backport PRs will be merged automatically after passing CI. To backport manually run: |
* [kbn-es/installers] convert to TypeScript * catch invalid urls passed to installArchive() # Conflicts: # packages/kbn-es/src/custom_snapshots.ts
* [kbn-es/installers] convert to TypeScript * catch invalid urls passed to installArchive() Co-authored-by: Spencer <[email protected]>
* [kbn-es/installers] convert to TypeScript * catch invalid urls passed to installArchive()
In an attempt to improve kbn-es startup and reduce some of the log noise on CI I'm constantly breaking things unexpectedly and finding it very challenging to refactor things. In an effort to make that easier I'd like to convert the module to TypeScript, starting with the installers and their dependencies.