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

Add in non-standard lsp request for resolving schemas on the client-side #395

Merged
merged 2 commits into from
Nov 26, 2020

Conversation

JPinkney
Copy link
Contributor

@JPinkney JPinkney commented Nov 20, 2020

Signed-off-by: Josh Pinkney [email protected]

What does this PR do?

This PR is the client-side request to redhat-developer/yaml-language-server#359. The client-side handles the schema downloading and sending the information back to the yaml language server

What issues does this PR fix or reference?

#107

Is it tested? How?

The only way I know how to test this is on the Red Hat VPN.

You'll need the server side changes: redhat-developer/yaml-language-server#359

Once you connect to the VPN these schemas should resolve correctly

"yaml.schemas": {
    "https://gitlab.cee.redhat.com/cpaas/service/-/raw/development/resources/schemas/release-root.json": "release.yml",
    "https://gitlab.cee.redhat.com/cpaas/service/-/raw/development/resources/schemas/product-root.json": "product.yml"
  }

yaml/registerVSCodeContentRequest

Signed-off-by: Josh Pinkney <[email protected]>
@JPinkney JPinkney merged commit b2560d1 into master Nov 26, 2020
@JPinkney JPinkney deleted the content-request branch November 26, 2020 14:16
bleach31 pushed a commit to bleach31/vscode-yaml that referenced this pull request Jan 25, 2022
* add code action to open json schema

Signed-off-by: Yevhen Vydolob <[email protected]>

* Fix eslint warning, fix review comments

Signed-off-by: Yevhen Vydolob <[email protected]>

* Fix tests

Signed-off-by: Yevhen Vydolob <[email protected]>

* simplify code actions implementation

Signed-off-by: Yevhen Vydolob <[email protected]>

* add code action to open schema only if 'showDocument' supported

Signed-off-by: Yevhen Vydolob <[email protected]>

* Fix tests

Signed-off-by: Yevhen Vydolob <[email protected]>

* fix test

Signed-off-by: Yevhen Vydolob <[email protected]>

* Add supported protocol version in readme

Signed-off-by: Yevhen Vydolob <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants