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

[Presentation Team] Migrate to Typescript Project References #86019

Merged
merged 10 commits into from
Jan 31, 2021

Conversation

clintandrewhall
Copy link
Contributor

@clintandrewhall clintandrewhall commented Dec 15, 2020

Summary

Applies to #80508

This is a draft PR to set up Typescript projects for:

  • canvas
  • input_control_vis
  • presentation_util
  • reporting

@clintandrewhall clintandrewhall added Feature:Dashboard Dashboard related features WIP Work in progress Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas loe:small Small Level of Effort release_note:skip Skip the PR/issue when compiling release notes impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. v7.12.0 Feature:Canvas labels Dec 15, 2020
@clintandrewhall clintandrewhall changed the title Initial commit; dependencies missing [Presentation Team] Migrate to Typescript Project References Dec 15, 2020
@clintandrewhall
Copy link
Contributor Author

@elasticmachine merge upstream

@clintandrewhall
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

merge conflict between base and head

@clintandrewhall
Copy link
Contributor Author

@elasticmachine merge upstream

@mshustov
Copy link
Contributor

@clintandrewhall are there any blockers to merge the PR?

@clintandrewhall clintandrewhall marked this pull request as ready for review January 29, 2021 14:42
@clintandrewhall clintandrewhall requested a review from a team as a code owner January 29, 2021 14:42
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

@clintandrewhall clintandrewhall added review and removed WIP Work in progress labels Jan 29, 2021
{ "path": "./src/plugins/data/tsconfig.json" },
{ "path": "./src/plugins/dev_tools/tsconfig.json" },
{ "path": "./src/plugins/discover/tsconfig.json" },
{ "path": "./src/plugins/embeddable/tsconfig.json" },
{ "path": "./src/plugins/es_ui_shared/tsconfig.json" },
{ "path": "./src/plugins/expressions/tsconfig.json" },
{ "path": "./src/plugins/home/tsconfig.json" },
{ "path": "./src/plugins/dashboard/tsconfig.json" },
{ "path": "./src/plugins/dev_tools/tsconfig.json" },
Copy link
Member

Choose a reason for hiding this comment

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

woah - nice job spotting a dup!

Copy link
Member

@tsullivan tsullivan left a comment

Choose a reason for hiding this comment

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

This is great! Much appreciated.

LGTM

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@clintandrewhall clintandrewhall merged commit af337ce into elastic:master Jan 31, 2021
@clintandrewhall clintandrewhall deleted the typescript-projects branch January 31, 2021 14:38
clintandrewhall added a commit to clintandrewhall/kibana that referenced this pull request Jan 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Canvas Feature:Dashboard Dashboard related features impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. loe:small Small Level of Effort release_note:skip Skip the PR/issue when compiling release notes review Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas v7.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants