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

Services typescripification #32038

Closed
wants to merge 2 commits into from

Conversation

vitalics
Copy link
Contributor

@vitalics vitalics commented Feb 26, 2019

Summary

Make autocomplete and type checking for services.

After merge this PR your IDE will help you with using services(see screen below)
image
Also it helps for existing servises usage(e.g. log)
image

For using this features need to change file extension to ts and write type for parametrs in wrap function(see screen below)
image
If you want to stay on js file extension you can use JSDoc workaround(see screen below)
image

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

- [ ] This was checked for cross-browser compatibility, including a check against IE11
- [ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
- [ ] Documentation was added for features that require explanation or tutorials
- [ ] Unit or functional tests were updated or added to match the most common scenarios
- [ ] This was checked for keyboard-only and screenreader accessibility

For maintainers

@vitalics vitalics added Team:QA Team label for QA Team Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Feb 26, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-qa

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app

@elasticmachine
Copy link
Contributor

💔 Build Failed

@vitalics
Copy link
Contributor Author

vitalics commented Mar 5, 2019

close this PR since #31948 was already merged in master branch

@vitalics vitalics closed this Mar 5, 2019
@vitalics vitalics deleted the services-typescripification branch March 5, 2019 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:QA Team label for QA Team Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants