Skip to content
This repository has been archived by the owner on Feb 1, 2019. It is now read-only.

.vscode-client: Drop vscode client #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ksdme
Copy link
Member

@ksdme ksdme commented Jun 8, 2018

Drop vscode-client since no component of coala-ls now is specific to or depends on VSCode.

@gaocegege
Copy link
Member

We are using this repository to place language server and coala/coala-vs-code to place the vs code client. Thus we need to remove the client from this repository and remove language server from coala/coala-vs-code after all things in this repository is ready.

LGTM

@gaocegege
Copy link
Member

/assign @jayvdb

I think we need you to merge the PR.

@jayvdb
Copy link
Member

jayvdb commented Jun 8, 2018

There are still two more references to vscode-client

$ git grep vscode-client
README.md:Then you should update the `./vscode-client/src/extension.ts` to make client in TCP mode.
README.md:To try it in [Visual Studio Code](https://code.visualstudio.com), open 

Those need to be removed to make this patch atomic.

@jayvdb jayvdb added the wip label Jun 8, 2018
@gaocegege
Copy link
Member

@ksdme

Maybe we could remove the vs code section in readme here instead of in the PR #8.

Drop vscode client since no component of
coala-ls now is specific to or depends
on VSCode.
@gaocegege
Copy link
Member

Could we merge it?

@gaocegege
Copy link
Member

@gitmate-bot ff

1 similar comment
@gaocegege
Copy link
Member

@gitmate-bot ff

@jayvdb
Copy link
Member

jayvdb commented Oct 11, 2018

This repo is going to be destroyed/archived and replaced with a clean rebuilt repo.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

3 participants