Skip to content

Update babel-typescript config#53108

Merged
patrykkopycinski merged 2 commits intoelastic:masterfrom
patrykkopycinski:chore/babel-typescript-namespaces
Dec 16, 2019
Merged

Update babel-typescript config#53108
patrykkopycinski merged 2 commits intoelastic:masterfrom
patrykkopycinski:chore/babel-typescript-namespaces

Conversation

@patrykkopycinski
Copy link
Contributor

@patrykkopycinski patrykkopycinski commented Dec 16, 2019

Summary

@babel/preset-typescript already supports namespaces, so we don't need extra plugin for removing them

Checklist

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

For maintainers

@patrykkopycinski patrykkopycinski requested a review from a team as a code owner December 16, 2019 10:57
@patrykkopycinski patrykkopycinski self-assigned this Dec 16, 2019
@patrykkopycinski patrykkopycinski added release_note:skip Skip the PR/issue when compiling release notes v7.6.0 v8.0.0 Team:Operations Kibana-Operations Team labels Dec 16, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Copy link
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@patrykkopycinski patrykkopycinski merged commit 4d50442 into elastic:master Dec 16, 2019
@patrykkopycinski patrykkopycinski deleted the chore/babel-typescript-namespaces branch December 16, 2019 18:28
patrykkopycinski added a commit to patrykkopycinski/kibana that referenced this pull request Dec 16, 2019
* Update babel-typescript config

* Add @babel/preset-react
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team v7.6.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants