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

[ftr] convert remaining JS to TS #35110

Merged
merged 1 commit into from
Apr 18, 2019
Merged

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Apr 15, 2019

In order to start working on additional FTR improvements, and to keep those PRs small, I'd like to convert the remaining bits over to TS. I didn't touch the mocha directory, because it's going to involve a lot of anys as it's mostly mucking with globals and overriding stuff in mocha, and it's going to be gone with #34591.

@elasticmachine

This comment has been minimized.

@elasticmachine

This comment has been minimized.

@elasticmachine

This comment has been minimized.

@elasticmachine

This comment has been minimized.

@elasticmachine

This comment has been minimized.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@vitalics
Copy link
Contributor

LGTM

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@spalger spalger merged commit 9122511 into elastic:master Apr 18, 2019
spalger pushed a commit to spalger/kibana that referenced this pull request Apr 18, 2019
spalger pushed a commit to spalger/kibana that referenced this pull request Apr 18, 2019
spalger pushed a commit that referenced this pull request Apr 18, 2019
@spalger
Copy link
Contributor Author

spalger commented Apr 18, 2019

7.x/7.1: 239f604
7.0: c142dd3

spalger pushed a commit that referenced this pull request Apr 19, 2019
Backports the following commits to 7.0:
 - [ftr] convert remaining JS to TS  (#35110)
@spalger spalger deleted the tsify/ftr/finish branch April 19, 2019 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants