Skip to content
This repository was archived by the owner on Apr 9, 2021. It is now read-only.

Bumpup update vscode-html-languageserver to 8637f10dd31c677acfe4c3a0e5f91c457aa7a00a #7

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

h-michael
Copy link

@h-michael h-michael commented Dec 31, 2019

vscode-html-languageserver's tsconfig changed.
It extends ../../shared.tsconfig.json that is https://github.com/microsoft/vscode/blob/master/extensions/shared.tsconfig.json now.
So I've changed submodule directory and build.js.
If this method is acceptable, put the same modifications in vscode-json-languageserver-bin and vscode-css-languageserver-bin.

@h-michael
Copy link
Author

@DeltaEvo gentle ping

vscode-html-languageserver's tsconfig changed.
 It extends `../../shared.tsconfig.json` that is
`https://github.com/microsoft/vscode/blob/master/extensions/shared.tsconfig.json` now.
So I copy and use shared.tsconfig.json. I don't want to keep up with changes in shared.tsconfig.json,
but I couldn't think of any other easy way.
The submodule directory has also been changed to match the relative path of extends in tsconfig.json.
@strawberry-choco
Copy link

Would be nice if the npm version gets updated after this gets merged too. It is two years old.

@CSRaghunandan
Copy link

While we are waiting for this pull request to be merged, @h-michael can you please provide me steps to compile your branch and generate a binary for use with my text editor?

Thanks

@h-michael h-michael changed the title Bumpup update vscode-html-languageserver to ed6dd995b1686b06536e3735189d3ef8780bfaaa Bumpup update vscode-html-languageserver to 8637f10dd31c677acfe4c3a0e5f91c457aa7a00a Mar 10, 2020
@h-michael
Copy link
Author

@CSRaghunandan you can build by npm run pack.

@h-michael
Copy link
Author

@aeschli Would you review this PR?

@aeschli
Copy link

aeschli commented Mar 10, 2020

@h-michael You need to ask a maintainer of this repo.

@DeltaEvo
Copy link
Contributor

I will review it, I'm just a little busy and I need to take time to test it

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

Successfully merging this pull request may close these issues.

5 participants