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

Remove @elastic/javascript-typescript-langserver #51115

Merged
merged 3 commits into from
Nov 21, 2019

Conversation

epixa
Copy link
Contributor

@epixa epixa commented Nov 19, 2019

This was only used by the code plugin, which was removed in 7.5.0.

@epixa epixa added Team:Operations Team label for Operations Team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.6.0 labels Nov 19, 2019
@elasticmachine
Copy link
Contributor

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

@epixa epixa requested a review from a team November 19, 2019 20:45
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Member

@mistic mistic left a comment

Choose a reason for hiding this comment

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

LGTM

This was only used by the code plugin, which was removed in 7.5.0.
@epixa epixa force-pushed the code-dep-javascript-typescript-langserver branch from ecabda3 to c6f9cd7 Compare November 20, 2019 20:27
@epixa
Copy link
Contributor Author

epixa commented Nov 20, 2019

I redid this commit because the yarn file conflicted with my other package removal change that I merged. I'll do these PRs one at a time going forward to avoid duplicate reviews.

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.

Love seeing all that red!

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@epixa
Copy link
Contributor Author

epixa commented Nov 21, 2019

I merged in the latest master just to minimize the change of any lock file wonkiness due to drift from the latest changes. No additional review is necessary unless you want to (or things blow up).

@elasticmachine
Copy link
Contributor

💔 Build Failed

@epixa
Copy link
Contributor Author

epixa commented Nov 21, 2019

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

💔 Build Failed

@epixa
Copy link
Contributor Author

epixa commented Nov 21, 2019

It looks like the CI failure was an infra issue rather than a test failure: https://kibana-ci.elastic.co/job/elastic+kibana+pipeline-pull-request/11287/execution/node/565/log/

@epixa
Copy link
Contributor Author

epixa commented Nov 21, 2019

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@epixa epixa merged commit 6f5491c into elastic:master Nov 21, 2019
@epixa epixa deleted the code-dep-javascript-typescript-langserver branch November 21, 2019 19:24
epixa added a commit to epixa/kibana that referenced this pull request Nov 21, 2019
This was only used by the code plugin, which was removed in 7.5.0.
epixa added a commit that referenced this pull request Nov 21, 2019
This was only used by the code plugin, which was removed in 7.5.0.
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 Team label for Operations Team v7.6.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants